Fix tests #152
Merged
Fix tests #152
Conversation
This file contains 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
Collaborator
tkmcmaster
commented
Sep 29, 2023
tkmcmaster
commented
- Removed unneeded vars
- Added mock restore
- Fixed grouping of tests into describe
- Added AGENT_APPLICATION_NAME similar to the controller name
- Removed unneeded comment
Pagan-Idel
approved these changes
Sep 29, 2023
tkmcmaster
added a commit
that referenced
this pull request
Sep 29, 2023
* Removed references to SystemTest (#134) * Added README instructions for building and link to examples (#136) * Zip all map (#138) * add ZipAllMap, and some doc comments and tests * add doctests to pr.sh --------- Co-authored-by: Zachery Olson <[email protected]> * Adds skipBody CLI argument - Skips Request and Response Body in Try Output (#140) * Cargo version upgrade * Updated the Guide to include the new available flag options for instructional purposes * Added the ability to include skipBody argument in TryConfig and modified output to honor new skip flag * included the new argument in the TmpTryConfig for testing purposes * change version upgrade back to original as requested * Removed unnessesary matches as requested * Changes made to output format as requested * Format changes as requested * format changes as requested by Rustfmt in the github checks * Manually fixed format errors * Fixed additional fmt failure after updating rust to 1.72 --------- Co-authored-by: Trevor McMaster <[email protected]> * Updated dependencies and fixed deprecations (#143) * Updated dependencies and fixed deprecations * Updated Wasm-pack to match the version in the 0.6.0 scripting branch * Example wasm tests (#147) * Added additional tests for all the example yaml files * Updated the README for the current changes * Open source PPaaS (#149) * Added initial common directory * Added open source agent code * Added initial controller check-in - Not fully working. Need to get monorepo set up * Removed references to familysearch.org * Added code to the pr script to build the hdr-histogram for the controller * Attempt to fix the common install/build issues in the controller * Added code to make a monorepo out of the common, agent, and controller * Added new PPaaS PR flow - Renamed the old flows to show what they are testing - Changed the guide PR to only monitor the guide files - Added new flow for the PPaaS common, agent, and controller * Updated the PR flow and added husky * Added the .env.test files we're missing for the github action * Changed all catch blocks to catch unknown - Added type checking around the S3 errors * Basic working build * Working version of next buildgit status! * Fixed storybook by removing reference to deprecated zlib * Removed references to fslocal.org examples * Removed husky since it's not needed on all check-ins * Removed duplication in pr script * Re-enabled the react build now that it's working * Fixed the Application Name for the controller for unit tests * Added code to generage a .env.local for the build * Changed the .env.local file to environment variables * Cleaned up the global script to improve performance and nyc output * Fixed the nyc merge * Changed the build environment files back to .env so it can be overridden - environment variables override all .env files. * Removed unneeded vars * Added mock restore * Fixed grouping of tests into describe * Added AGENT_APPLICATION_NAME similar to the controller name * Removed unneeded comment * Removed additional unneeded vars * Fix tests (#152) * Removed unneeded vars * Added mock restore * Fixed grouping of tests into describe * Added AGENT_APPLICATION_NAME similar to the controller name * Removed unneeded comment * Fixed cargo lock file * Fixed duplication and issues from merge * Fixed merge change --------- Co-authored-by: Zachery Olson <[email protected]> Co-authored-by: Idel <[email protected]>
tkmcmaster
added a commit
that referenced
this pull request
Oct 2, 2023
* Removed references to SystemTest (#134) * Added README instructions for building and link to examples (#136) * Zip all map (#138) * add ZipAllMap, and some doc comments and tests * add doctests to pr.sh --------- Co-authored-by: Zachery Olson <[email protected]> * Adds skipBody CLI argument - Skips Request and Response Body in Try Output (#140) * Cargo version upgrade * Updated the Guide to include the new available flag options for instructional purposes * Added the ability to include skipBody argument in TryConfig and modified output to honor new skip flag * included the new argument in the TmpTryConfig for testing purposes * change version upgrade back to original as requested * Removed unnessesary matches as requested * Changes made to output format as requested * Format changes as requested * format changes as requested by Rustfmt in the github checks * Manually fixed format errors * Fixed additional fmt failure after updating rust to 1.72 --------- Co-authored-by: Trevor McMaster <[email protected]> * Updated dependencies and fixed deprecations (#143) * Updated dependencies and fixed deprecations * Updated Wasm-pack to match the version in the 0.6.0 scripting branch * Example wasm tests (#147) * Added additional tests for all the example yaml files * Updated the README for the current changes * Open source PPaaS (#149) * Added initial common directory * Added open source agent code * Added initial controller check-in - Not fully working. Need to get monorepo set up * Removed references to familysearch.org * Added code to the pr script to build the hdr-histogram for the controller * Attempt to fix the common install/build issues in the controller * Added code to make a monorepo out of the common, agent, and controller * Added new PPaaS PR flow - Renamed the old flows to show what they are testing - Changed the guide PR to only monitor the guide files - Added new flow for the PPaaS common, agent, and controller * Updated the PR flow and added husky * Added the .env.test files we're missing for the github action * Changed all catch blocks to catch unknown - Added type checking around the S3 errors * Basic working build * Working version of next buildgit status! * Fixed storybook by removing reference to deprecated zlib * Removed references to fslocal.org examples * Removed husky since it's not needed on all check-ins * Removed duplication in pr script * Re-enabled the react build now that it's working * Fixed the Application Name for the controller for unit tests * Added code to generage a .env.local for the build * Changed the .env.local file to environment variables * Cleaned up the global script to improve performance and nyc output * Fixed the nyc merge * Changed the build environment files back to .env so it can be overridden - environment variables override all .env files. * Fix tests (#152) * Removed unneeded vars * Added mock restore * Fixed grouping of tests into describe * Added AGENT_APPLICATION_NAME similar to the controller name * Removed unneeded comment * Update node dependencies (#154) * Updated lock files and a few npm dependencies * More dependencies updates - Rolled back dotenv-flow due to breaking change with importing config - Updated fullcalendar now that the next.js issue appears fixed * Updated the zip yaml files with the updated yaml files * Updated Next.js to latest version * Fixed merge duplicate test error --------- Co-authored-by: Zachery Olson <[email protected]> Co-authored-by: Idel <[email protected]>
tkmcmaster
added a commit
that referenced
this pull request
Oct 4, 2023
* Removed references to SystemTest (#134) * Added README instructions for building and link to examples (#136) * Zip all map (#138) * add ZipAllMap, and some doc comments and tests * add doctests to pr.sh --------- Co-authored-by: Zachery Olson <[email protected]> * Adds skipBody CLI argument - Skips Request and Response Body in Try Output (#140) * Cargo version upgrade * Updated the Guide to include the new available flag options for instructional purposes * Added the ability to include skipBody argument in TryConfig and modified output to honor new skip flag * included the new argument in the TmpTryConfig for testing purposes * change version upgrade back to original as requested * Removed unnessesary matches as requested * Changes made to output format as requested * Format changes as requested * format changes as requested by Rustfmt in the github checks * Manually fixed format errors * Fixed additional fmt failure after updating rust to 1.72 --------- Co-authored-by: Trevor McMaster <[email protected]> * Updated dependencies and fixed deprecations (#143) * Updated dependencies and fixed deprecations * Updated Wasm-pack to match the version in the 0.6.0 scripting branch * Example wasm tests (#147) * Added additional tests for all the example yaml files * Updated the README for the current changes * Open source PPaaS (#149) * Added initial common directory * Added open source agent code * Added initial controller check-in - Not fully working. Need to get monorepo set up * Removed references to familysearch.org * Added code to the pr script to build the hdr-histogram for the controller * Attempt to fix the common install/build issues in the controller * Added code to make a monorepo out of the common, agent, and controller * Added new PPaaS PR flow - Renamed the old flows to show what they are testing - Changed the guide PR to only monitor the guide files - Added new flow for the PPaaS common, agent, and controller * Updated the PR flow and added husky * Added the .env.test files we're missing for the github action * Changed all catch blocks to catch unknown - Added type checking around the S3 errors * Basic working build * Working version of next buildgit status! * Fixed storybook by removing reference to deprecated zlib * Removed references to fslocal.org examples * Removed husky since it's not needed on all check-ins * Removed duplication in pr script * Re-enabled the react build now that it's working * Fixed the Application Name for the controller for unit tests * Added code to generage a .env.local for the build * Changed the .env.local file to environment variables * Cleaned up the global script to improve performance and nyc output * Fixed the nyc merge * Changed the build environment files back to .env so it can be overridden - environment variables override all .env files. * Fix tests (#152) * Removed unneeded vars * Added mock restore * Fixed grouping of tests into describe * Added AGENT_APPLICATION_NAME similar to the controller name * Removed unneeded comment * Update node dependencies (#154) * Updated lock files and a few npm dependencies * More dependencies updates - Rolled back dotenv-flow due to breaking change with importing config - Updated fullcalendar now that the next.js issue appears fixed * Updated the zip yaml files with the updated yaml files * Updated Next.js to latest version * Updates from controller save master branch * Updated version and dependencies * Removed old command from README * Removed references to fsdpt.org * Removed unneeded vars and added missing ones * Updates on master PPaaS (#156) * Updated version and dependencies * Removed old command from README * Removed references to fsdpt.org * Removed unneeded vars and added missing ones * Updated YamlParser and tests - Updated config-wasm tests to test validate_legacy_only true and false - Updated yamlparser tests to test validate_legacy_only true and false - Updated yamlparser to pass optional parameter validate_legacy_only - Updated yamlparser to parse missing environment variables, bad durations, and bad hit rates for the new scripting config parser. * Updated tests for legacy and scripting - POST pewpew now checks for pre-release in the regex - Renamed numberedVersion to legacyVersion to more easily differentiate with scriptingVersion * Added scripting post pewpew tests to unit and integration * Changed agent version greaterthan to use semver * Added code to force validation of legacy or scritping to the yaml parser * Updated TestManager to validate yaml based on passed pewpew version - parseYamlFile will validate as legacy on any versions before 0.6.0, or as scripting on any >0.6.0-preview - no version or version latest will parse as any * Enabled acceptance tests that check against numbered versions of pewpew * Removed hardcoded latest string * Initial rename of variables and postTest of legacy against scripting version * Updated tests to include scripting variants in acceptance and integration post /test * Added test to test hidden variables not present on editSchedule --------- Co-authored-by: Zachery Olson <[email protected]> Co-authored-by: Idel <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.