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

Snapshot location #1489

Merged
merged 16 commits into from
Sep 2, 2017
Merged

Snapshot location #1489

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
f2471c8
Add ability for a custom snapshot location and resolving of the snaps…
impaler Jul 21, 2017
2c344fa
#1459 Resolve custom location for non sourcemap tests.
impaler Aug 7, 2017
5c65c9e
Add snapshot integration tests for location & sourcemap features #1459
impaler Aug 7, 2017
9e98219
fix ava config #1459
impaler Aug 7, 2017
548c6b6
(doc) snapshotLocation configuration and sourcemap support in typescr…
impaler Aug 13, 2017
d239882
Add tsconfig.json to reproduce sourcemap snapshot fixture.
impaler Aug 13, 2017
12f1ff8
(refactor) based on feedback, now uses 'convert-source-map' #1489
impaler Aug 16, 2017
902f0d3
(style) fix indentation, remove build script.
impaler Aug 16, 2017
3ec0217
(style) fix indentation.
impaler Aug 23, 2017
e3fc508
(refactor) update snapshot location integration tests based on feedba…
impaler Aug 23, 2017
6282999
Keep snapshot manager options in alphabetical order
novemberborn Aug 27, 2017
de453b3
Add fixedLocation option when loading snapshot manager in tests
novemberborn Aug 27, 2017
22838a9
Use path.relative() rather than string-replacing path prefixes
novemberborn Aug 27, 2017
b18db03
Refactor source-mapped directory determination
novemberborn Aug 27, 2017
61af5da
Touch up README
novemberborn Aug 27, 2017
982a0a2
Correct AVA capitalization in test title
novemberborn Aug 27, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use path.relative() rather than string-replacing path prefixes
  • Loading branch information
@novemberborn
novemberborn committed Aug 27, 2017
commit 22838a970cc65a9854ee2652e09da0e646c92596
2 changes: 1 addition & 1 deletion lib/snapshot-manager.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -359,7 +359,7 @@ class Manager {
function determineSnapshotDir(options) {
const testDir = determineSourceMappedDir(options);
if (options.fixedLocation) {
const relativeTestLocation = testDir.replace(options.projectDir, '');
const relativeTestLocation = path.relative(options.projectDir, testDir);
return path.join(options.fixedLocation, relativeTestLocation);
}
const parts = new Set(path.relative(options.projectDir, testDir).split(path.sep));
Expand Down