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.

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

Run fragment-check when internal flag is true #153

Merged
merged 5 commits into from
Jul 21, 2019

Conversation

Munter
Copy link
Owner

@Munter Munter commented Jul 21, 2019

Fixes #152

When running with the internal flag or --internal in the CLI, fragment checks were not run at all. Also not for completely site internal fragment links.

@Munter Munter force-pushed the fix/issue-152-internal-fragments-not-checked branch from 03ea714 to c23bf21 Compare July 21, 2019 12:20
@Munter Munter force-pushed the fix/issue-152-internal-fragments-not-checked branch from c23bf21 to ad99c47 Compare July 21, 2019 12:23
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 77.966% when pulling 0245a66 on fix/issue-152-internal-fragments-not-checked into f146039 on master.

@coveralls
Copy link

coveralls commented Jul 21, 2019

Coverage Status

Coverage decreased (-0.2%) to 77.966% when pulling f47fe2f on fix/issue-152-internal-fragments-not-checked into f146039 on master.

Copy link
Collaborator

@papandreou papandreou left a comment

Choose a reason for hiding this comment

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

Good catch!

@@ -441,13 +449,68 @@ describe('hyperlink', function() {
t
);

expect(spyTapCalls(t.push), 'with set semantics to satisfy', [
Copy link
Collaborator

Choose a reason for hiding this comment

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

As I mentioned in https://gitter.im/unexpectedjs/unexpected?at=5d3475f2ec5abe7bbc227eab you should be able to use to have calls satisfying here.

Co-Authored-By: Andreas Lind <[email protected]>
@Munter Munter merged commit bfd03fd into master Jul 21, 2019
@Munter Munter deleted the fix/issue-152-internal-fragments-not-checked branch July 21, 2019 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fragment identifier check does not run with --internal active
3 participants