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

CircleCI tests emit a false positive result #150

Open
cantsin opened this issue May 1, 2020 · 2 comments
Open

CircleCI tests emit a false positive result #150

cantsin opened this issue May 1, 2020 · 2 comments

Comments

@cantsin
Copy link
Contributor

cantsin commented May 1, 2020

When running CircleCI tests, I noticed that tests (specifically htmlproofer) are running on an empty directory and thereby always succeeding.

Example run:

Running ["ScriptCheck", "HtmlCheck", "LinkCheck", "ImageCheck"] on ["_site"] on *.html... 
Ran on 0 files!
HTML-Proofer finished successfully.

To fix this, we want to persist the generated files in the CircleCI config, e.g.:

      - run:
          name: test
          command: npm test

      - persist_to_workspace:
          root: ./
          paths:
            - _site

Unfortunately, for this to work the baseurl needs to be set to "". Once #148 is merged (and there is no need for a preview url prefix anymore), we can remove baseurl and insert the above fix to make tests work again. (Note: #148 fixes all currently broken links)

@adborden
Copy link
Contributor

Just noticed this. It's not about the persist_to_workspace, it's that jekyll build is never called within CI.

@adborden adborden mentioned this issue May 15, 2020
1 task
@adborden
Copy link
Contributor

This is how I would fix it ^. We can merge #148 before merging the CI fix.

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

No branches or pull requests

2 participants