Skip to content

Commit

Permalink
update to frodo-lib 2.0.0-77 and remove secrets from recordings
Browse files Browse the repository at this point in the history
  • Loading branch information
vscheuber committed Apr 9, 2024
1 parent 5132902 commit 512988f
Show file tree
Hide file tree
Showing 34 changed files with 112,243 additions and 78,480 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Changed

- Update to frodo-lib 2.0.0-76
- Update to frodo-lib 2.0.0-77

### Fixed

- Improved filtering out secrets from recordings
- rockcarver/frodo-lib#392: Implemented error handling pattern for methods with unusual amounts of REST calls like `frodo.config.exportFullConfiguration` and `frodo.config.importFullConfiguration` used in the `frodo config import` and `frodo config export` commands

## [2.0.0-54] - 2024-04-01
Expand Down
8 changes: 4 additions & 4 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@
]
},
"dependencies": {
"@rockcarver/frodo-lib": "2.0.0-76",
"@rockcarver/frodo-lib": "2.0.0-77",
"chokidar": "^3.5.3",
"cli-progress": "^3.11.2",
"cli-table3": "^0.6.3",
Expand Down
76,760 changes: 43,405 additions & 33,355 deletions test/e2e/__snapshots__/config-export.e2e.test.js.snap

Large diffs are not rendered by default.

2 changes: 0 additions & 2 deletions test/e2e/__snapshots__/config-import.e2e.test.js.snap
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,6 @@ Errors occurred during full config import
Status: 401
Reason: Unauthorized
Message: Unauthorized
[========================================] 100% | 16/16 | Finished Importing Everything!
[========================================] 100% | 22/22 | Finished importing journeys
"
`;

Expand Down
4 changes: 3 additions & 1 deletion test/e2e/config-import.e2e.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,9 @@ describe('frodo config import', () => {
await exec(CMD, env);
fail("Command should've failed")
} catch (e) {
expect(removeAnsiEscapeCodes(e.stderr)).toMatchSnapshot();
// parallel test execution alters the progress bar output causing the snapshot to mismatch.
// only workable solution I could find was to remove progress bar output altogether from such tests.
expect(removeProgressBarOutput(removeAnsiEscapeCodes(e.stderr))).toMatchSnapshot();
}
});

Expand Down
Loading

0 comments on commit 512988f

Please sign in to comment.