Skip to content

WEB-339: add testing harness to 'primo-explore-bu' packages/#23

Open
atla5 wants to merge 12 commits into
masterfrom
WEB-339
Open

WEB-339: add testing harness to 'primo-explore-bu' packages/#23
atla5 wants to merge 12 commits into
masterfrom
WEB-339

Conversation

@atla5

@atla5 atla5 commented Apr 14, 2020

Copy link
Copy Markdown
Contributor

Jira Ticket: WEB-339

Background

Description of Changes

  • add dependencies and harness for running jasmine tests with karma
  • create smoke tests for each package that effectively loads each module
  • get a basic npm run start going to best enable users to test-run the changes locally

atla5 added 2 commits April 13, 2020 15:32
- getting multiple errors in help-menu
```
Error: [$injector:modulerr] Failed to instantiate module helpMenuTopbar due to:
[$injector:modulerr] Failed to instantiate module ngMaterial due to:
[$injector:nomod] Module 'ngMaterial' is not available!
```

- and an unknown reference in unpaywall to 'bulibUnpaywallProvider'
```
Error: [$injector:unpr] Unknown provider: bulibUnpaywallProvider <- bulibUnpaywall
```
@atla5 atla5 added help wanted Extra attention is needed in progress work is actively underway towards completion labels Apr 14, 2020
atla5 added 4 commits April 14, 2020 13:10
- still experiencing the 'Provider' errors, though:

```
Error: [$injector:unpr] Unknown provider: helpMenuContentDisplayProvider <- helpMenuContentDisplay
...
Error: [$injector:unpr] Unknown provider: bulibUnpaywallProvider <- bulibUnpaywall
```
- still haven't fixed the injector
Comment thread packages/unpaywall/tests/unpaywall.spec.js Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed in progress work is actively underway towards completion

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants