-
Notifications
You must be signed in to change notification settings - Fork 15
Merge dev into main (Sprint 5) #24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
fe10e73
Implement test orchestrator (#4)
jhughesoti 6f3a7fe
Add issue report templates (#7)
jboddey e05c383
Discover devices on the network (#5)
jboddey 823709e
Test run sync (#8)
jhughesoti ba6afc4
Quick refactor (#9)
jboddey c87a976
Fix duplicate sleep calls
jhughesoti 34ce211
Add net orc (#11)
jhughesoti ceba453
Add the DNS test module (#12)
jhughesoti 0837a9c
Add baseline and pylint tests (#25)
noursaidi 4171e5f
Discover devices on the network (#22)
jboddey be829a3
Build dependencies first (#21)
jboddey 84d9ff9
Port scan test module (#23)
jhughesoti 07432ee
Fix device configs
jboddey 7b27e23
Remove unecessary files
jhughesoti 5ac8726
Cleanup duplicate properties
jhughesoti 2c4efe8
Cleanup install script
jhughesoti 25fd8a5
Formatting (#26)
jhughesoti 41aaaf7
Test results (#27)
jhughesoti ea60b41
Test results (#28)
jhughesoti b6a6cdc
Fix pylint test and skip internet tests so CI passes (#29)
noursaidi 3d53ecb
Increase pylint score (#31)
jboddey b91fff5
Pylint (#32)
jhughesoti File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| --- | ||
| name: Bug report | ||
| about: Create a report to help us identify and resolve bugs | ||
| title: '' | ||
| labels: bug | ||
| assignees: '' | ||
|
|
||
| --- | ||
|
|
||
| **Describe the bug** | ||
| A clear and concise description of what the bug is. | ||
|
|
||
| **To Reproduce** | ||
| Steps to reproduce the behavior: | ||
| 1. Go to '...' | ||
| 2. Click on '....' | ||
| 3. Scroll down to '....' | ||
| 4. See error | ||
|
|
||
| **Expected behavior** | ||
| A clear and concise description of what you expected to happen. | ||
|
|
||
| **Error logs** | ||
| If applicable, provide a log from https://gist.github.com/ | ||
|
|
||
| **Environment (please provide the following information about your setup):** | ||
| - OS: [e.g. Ubuntu] | ||
| - Version [e.g. 22.04] | ||
| - Additional hardware (network adapters) | ||
|
|
||
| **Additional context** | ||
| Add any other context about the problem here. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| --- | ||
| name: Feature request | ||
| about: Suggest a new feature or change request | ||
| title: '' | ||
| labels: request | ||
| assignees: '' | ||
|
|
||
| --- | ||
|
|
||
| **What is the problem your feature is trying to solve?** | ||
| A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] | ||
|
|
||
| **Describe the solution you think would solve the problem** | ||
| A clear and concise description of what you want to happen. | ||
|
|
||
| **Additional context** | ||
| Add any other context or screenshots about the feature request here. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| name: Testrun test suite | ||
|
|
||
| on: | ||
| push: | ||
| pull_request: | ||
| schedule: | ||
| - cron: '0 13 * * *' | ||
|
|
||
| jobs: | ||
| testrun: | ||
| name: Baseline | ||
| runs-on: ubuntu-20.04 | ||
jboddey marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| timeout-minutes: 20 | ||
| steps: | ||
| - name: Checkout source | ||
| uses: actions/checkout@v2.3.4 | ||
| - name: Run tests | ||
| shell: bash {0} | ||
| run: testing/test_baseline | ||
|
|
||
| pylint: | ||
| name: Pylint | ||
| runs-on: ubuntu-20.04 | ||
| timeout-minutes: 20 | ||
| steps: | ||
| - name: Checkout source | ||
| uses: actions/checkout@v2.3.4 | ||
| - name: Run tests | ||
| shell: bash {0} | ||
| run: testing/test_pylint | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,7 @@ | ||
| venv/ | ||
| net_orc/ | ||
| .vscode/ | ||
| runtime/ | ||
| venv/ | ||
| .vscode/ | ||
| error | ||
| pylint.out | ||
| local/ | ||
| __pycache__/ |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@noursaidi Is there anyway we can stop both of these running on pull requests? It is running twice on the PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry no notifications. Yes, will investigate