Skip to content
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.

Sign up for GitHub

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

Jump to bottom

https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1137 : Renaming tests to be consistent, removing old tests, adding README.md for tests. #822

Merged
merged 58 commits into from
Aug 10, 2021
Merged

https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1137 : Renaming tests to be consistent, removing old tests, adding README.md for tests. #822

merged 58 commits into from
Aug 10, 2021

Conversation

JackAtGaia
Copy link

@JackAtGaia JackAtGaia commented Aug 10, 2021 •
edited
Loading

With the exception of a 1-line change to allow blank lines in test suite files, the rest of this PR is all about cleaning up the tests and suites.

  • two tests were no longer being used, and they were deleted
  • the names of the base tests has changed, so changed the *-1k, *-10k etc tests to use the new name
  • each test gets its own README.md
  • added blank line support for suite files to allow for cleaner looking files

p.s. I have no clue why this PR says there are 58 commits in it... there are 2-3 at most.

JackAtGaia added 30 commits July 7, 2021 15:31
@JackAtGaia
inital submission
563a06c
@JackAtGaia
Cleaning up main test script.
81b06cf
@JackAtGaia
Adding capturing of gaia_stats.log
5e27352
@JackAtGaia
Cleaning up run.sh and making CSV generation optional.
c7dca4f
@JackAtGaia
Adding code to allow specific test source information.
3c00f7d
@JackAtGaia
Cleaning install.sh up.
d1a92df
@JackAtGaia
Finishing up initial pass on standardizing scripts.
da5457c
@JackAtGaia
Making scripts more generic by pulling stuff into properties.h
3db99b3
@JackAtGaia
Making all scripts as independant of the project as possible.
603f6d7
@JackAtGaia
First pass at adding suite support.
934a4ae
@JackAtGaia
Adding summary.json to capture a summary of the suite test.
c4e2ffc
@JackAtGaia
Integrating gaia.conf into tests.
b6666ea
@JackAtGaia
Refining pause between iterations in debug.
3f13073
@JackAtGaia
Adding new tests for 100 command and 100K commands.
2135462
@JackAtGaia
First attempt at stabilizing tests.
80dd121
@JackAtGaia
Adding generated configuration file.
256569d
@JackAtGaia
Moved Python scripts into own directory.
b20d5a6
@JackAtGaia
Added calculation of averages and maximum values for suites.
650ba54
@JackAtGaia
Adding list options for suite.sh and test.sh
7d69594
@JackAtGaia
Taking care of properly showing the suite names.
1ede94a
@JackAtGaia
Cleaned up tests to make them stable, and renamed to provide proper c…
fa4cfd1 
…ontext.
@JackAtGaia
Adding extra measurements to account for more time.
92ee4d0
@JackAtGaia
added support for running tests multiple times.
3803b95
@JackAtGaia
Small bits of cleanup.
0b09d32
@JackAtGaia
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1081 : Capturing c…
92d2a12 
…onfiguration in test runs.
@JackAtGaia
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1090 : Adding slac…
a9ea27f 
…k support.
@JackAtGaia
Adding copyright notices to Python and Bash scripts.
aa135d4
@JackAtGaia
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1097 : Fixing smal…
3536fa1 
…l errors found during documentation.
@JackAtGaia
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1097 : First pass …
faeab52 
…at new documentation.
@JackAtGaia
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1097 - First solid…
6248254 
… pass at new readme.md.
JackAtGaia added 14 commits July 29, 2021 13:49
@JackAtGaia
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1122 : Adding obse…
bf57c17 
…rvations and fixes per "smoke"
@JackAtGaia
Updated "smoke" common commands to take observed time, plus small cle…
6f594f3 
…anups.
@JackAtGaia
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1093 : Added abili…
d37a0c2 
…ty to specify thread count as part of the suite test definition.
@JackAtGaia
Adding extra command 'x' to make sure that no more rules are firing. …
17027df 
…for use at end of script if using 's[num]'
@JackAtGaia
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1128 : Adding firs…
f2ad947 
…t draft.
@JackAtGaia
Saving state before installing new build.
875f3cd
@JackAtGaia
Moving code to test directory before merge with main repo.
6a3d832
@JackAtGaia
Merge remote-tracking branch 'test/master' into merge-test
6fcef1a
@JackAtGaia
Resolving small errors after moving of test into main repository.
387ee56
@JackAtGaia
#817 : Responding to comments from Simone.
fff24c2
@JackAtGaia
#817 : Extra wordsmithing per SimoneR
faf03f7
@JackAtGaia
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1137 : Cleaning up…
42c2d84 
… test definitions and suite definitions.
@JackAtGaia
Merge branch 'master' of https://github.com/gaia-platform/GaiaPlatform …
b16ae4b 
…into merge-test
@JackAtGaia
Missed merge files.
d7243ee
@JackAtGaia JackAtGaia changed the title Merge test https://gaiaplatform.atlassian.net/browse/GAIAPLAT-1137 : Renaming tests to be consistent, removing old tests, adding README.md for tests. Aug 10, 2021
@JackAtGaia JackAtGaia requested a review from daxhaw August 10, 2021 21:57
@JackAtGaia JackAtGaia self-assigned this Aug 10, 2021
@JackAtGaia JackAtGaia marked this pull request as ready for review August 10, 2021 21:58
simone-gaia
simone-gaia approved these changes Aug 10, 2021
View reviewed changes
Copy link
Contributor

@simone-gaia simone-gaia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@JackAtGaia JackAtGaia merged commit 506c5e6 into master Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simone-gaia simone-gaia simone-gaia approved these changes

@daxhaw daxhaw Awaiting requested review from daxhaw

@chuan chuan Awaiting requested review from chuan

@fineg74 fineg74 Awaiting requested review from fineg74

@mihirj1993 mihirj1993 Awaiting requested review from mihirj1993

@phillipov phillipov Awaiting requested review from phillipov

@senderista senderista Awaiting requested review from senderista

@tengizk tengizk Awaiting requested review from tengizk

@waynelwarren waynelwarren Awaiting requested review from waynelwarren

@yiwen-wong yiwen-wong Awaiting requested review from yiwen-wong

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JackAtGaia @simone-gaia