Skip to content

Moved test_group files to profile folder, fixes #670#671

Merged
DePierre merged 1 commit intoowtf:developfrom
DarKnight24:fix_670
Apr 22, 2016
Merged

Moved test_group files to profile folder, fixes #670#671
DePierre merged 1 commit intoowtf:developfrom
DarKnight24:fix_670

Conversation

@DarKnight24
Copy link
Copy Markdown
Contributor

Moved test_group files to profile folder

Related Issue

fixes #670

Motivation and Context

The changes will separate the plugin configuration files from framework.

Reviewers

@DePierre

How Has This Been Tested?

./owtf.py -l auxiliary, lists successfully the plugins of the auxiliary group.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Other

Checklist:

  • My code follows the code style (modified PEP8) of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@viyatb
Copy link
Copy Markdown
Member

viyatb commented Apr 21, 2016

@DarKnight-- Better make that testgroups folder (in sync with the filenames aux_testgroups)

@DePierre
Copy link
Copy Markdown
Contributor

@delta24 How about moving them in the same folder as net_plugin_order, etc. ?

We could rename the directories to something like:

  • plugin_web
    • plugin_web_order.cfg
    • plugin_web_test_groups.cfg

The same for aux and net. What do you think?

@viyatb
Copy link
Copy Markdown
Member

viyatb commented Apr 21, 2016

@DePierre sure, nice idea!. Just that the plugin_web_order.cfg should be web_order.cfg, etc. :P (repetitive)

@DePierre
Copy link
Copy Markdown
Contributor

Then why not just order.cfg and groups.cfg since web would be repetitive as well?
To be honest, I am not even sure why it is named testgroups.cfg in the first place, do you know @delta24 ?

@DePierre
Copy link
Copy Markdown
Contributor

Trigger Travis.

@DePierre DePierre closed this Apr 22, 2016
@DePierre DePierre reopened this Apr 22, 2016
@DePierre DePierre closed this Apr 22, 2016
@DePierre DePierre reopened this Apr 22, 2016
@DePierre DePierre merged commit cfdeefb into owtf:develop Apr 22, 2016
@DePierre
Copy link
Copy Markdown
Contributor

@DarKnight-- Thank you! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

xxx_testgroups.cfg should be moved to /profiles

3 participants