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

Testing #35

Merged
merged 26 commits into from
Oct 17, 2019
Merged

Testing #35

merged 26 commits into from
Oct 17, 2019

Conversation

bilalshaikh42
Copy link
Member

Fix for #9
Fixes all tests to make them pass
Does not actually include needed tests for all components

@todo
Copy link

todo bot commented Oct 16, 2019

@bilalshaikh42 bilalshaikh42 added this to High priority in Bugs Oct 16, 2019
@todo
Copy link

todo bot commented Oct 16, 2019

@todo
Copy link

todo bot commented Oct 16, 2019

@todo
Copy link

todo bot commented Oct 16, 2019

@bilalshaikh42
Copy link
Member Author

this test is failing due to error in component

https://github.com/reproducible-biomedical-modeling/CRBM-Viz/blob/98fdcfa37ff2f487d478d8157a3307f359f4ad3a/CRBM-Viz/src/app/Pages/files/file-edit/file-edit.component.spec.ts#L11-L16

This comment was generated by todo based on a TODO comment in 98fdcfa in #35. cc @reproducible-biomedical-modeling.

@gmarupilla Could you take a look at the tests for this component?

@gmarupilla
Copy link
Contributor

this test is failing due to error in component

https://github.com/reproducible-biomedical-modeling/CRBM-Viz/blob/98fdcfa37ff2f487d478d8157a3307f359f4ad3a/CRBM-Viz/src/app/Pages/files/file-edit/file-edit.component.spec.ts#L11-L16

This comment was generated by todo based on a TODO comment in 98fdcfa in #35. cc @reproducible-biomedical-modeling.

@gmarupilla Could you take a look at the tests for this component?

Let me look into that!

@todo
Copy link

todo bot commented Oct 16, 2019

@todo
Copy link

todo bot commented Oct 16, 2019

Sample content is needed and component compiled with it as input for <ng-content>

https://github.com/reproducible-biomedical-modeling/CRBM-Viz/blob/b796175f212350071b0510c7fd686e7c201e5c71/CRBM-Viz/src/app/Layout/navigation/navigation.component.spec.ts#L13-L18


This comment was generated by todo based on a TODO comment in b796175 in #35. cc @reproducible-biomedical-modeling.

@todo
Copy link

todo bot commented Oct 16, 2019

@todo
Copy link

todo bot commented Oct 16, 2019

@todo
Copy link

todo bot commented Oct 16, 2019

@todo
Copy link

todo bot commented Oct 16, 2019

Sample content is needed and component compiled with it as input for <ng-content>

https://github.com/reproducible-biomedical-modeling/CRBM-Viz/blob/23c5842c737de9d4b704ccd5b645a5500ff32d47/CRBM-Viz/src/app/Layout/navigation/navigation.component.spec.ts#L13-L18


This comment was generated by todo based on a TODO comment in 23c5842 in #35. cc @reproducible-biomedical-modeling.

@bilalshaikh42 bilalshaikh42 marked this pull request as ready for review October 16, 2019 22:36
@todo
Copy link

todo bot commented Oct 17, 2019

Sample content is needed and component compiled with it as input for <ng-content>

https://github.com/reproducible-biomedical-modeling/CRBM-Viz/blob/c191d8554fc07a167d9df0e155c5aa1ea9e430e0/CRBM-Viz/src/app/Layout/navigation/navigation.component.spec.ts#L13-L18


This comment was generated by todo based on a TODO comment in c191d85 in #35. cc @reproducible-biomedical-modeling.

@todo
Copy link

todo bot commented Oct 17, 2019

this test causes the next test to fail due to missing alertcomponent factory

https://github.com/reproducible-biomedical-modeling/CRBM-Viz/blob/9990812697ca6b0d1c64faa445e907e7e1b21055/CRBM-Viz/src/app/Components/new-simulation/new-simulation.component.spec.ts#L12-L17


This comment was generated by todo based on a TODO comment in 9990812 in #35. cc @reproducible-biomedical-modeling.

@todo
Copy link

todo bot commented Oct 17, 2019

this test causes the next test to fail due to missing alertcomponent factory

https://github.com/reproducible-biomedical-modeling/CRBM-Viz/blob/9990812697ca6b0d1c64faa445e907e7e1b21055/CRBM-Viz/src/app/Components/past-simulation/past-simulation.component.spec.ts#L11-L16


This comment was generated by todo based on a TODO comment in 9990812 in #35. cc @reproducible-biomedical-modeling.

@todo
Copy link

todo bot commented Oct 17, 2019

Sample content is needed and component compiled with it as input for <ng-content>

https://github.com/reproducible-biomedical-modeling/CRBM-Viz/blob/9990812697ca6b0d1c64faa445e907e7e1b21055/CRBM-Viz/src/app/Layout/navigation/navigation.component.spec.ts#L13-L18


This comment was generated by todo based on a TODO comment in 9990812 in #35. cc @reproducible-biomedical-modeling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Bugs
  
Closed
Development

Successfully merging this pull request may close these issues.

None yet

2 participants