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

20826 Super setup need to be called in DependencyAnalyser tests #579

Conversation

astares
Copy link
Member

@astares astares commented Dec 6, 2017

  • call super setUp in DADependencyCheckerTest
  • call super setUp in DAMessageSendAnalyzerTest
  • call super setUp in DAPackageCycleDetectorTest (with some formatting cleanups in setUp)
  • call super setUp in DAPackageCycleTest
  • call super setUp in DAPackageDependencyTest
  • call super setUp in DAPackageDependencyWrapperTest
  • call super setUp in DAPackageRelationGraphDiffTest
  • call super setUp in DAPackageRelationGraphTest (with some formatting cleanups in setUp)
  • call super setUp in DAPackageTest (with some formatting cleanups in setUp)
  • call super setUp in DATarjanAlgorithmTest (with some formatting cleanups in setUp)
    Also add some tags to to better categorize the test classes in package "Tool-Dependency-Analyse-Test"

- call super setUp in DAMessageSendAnalyzerTest
- call super setUp in DAPackageCycleDetectorTest (with some formatting cleanups in setUp)
- call super setUp in DAPackageCycleTest
- call super setUp in DAPackageDependencyTest
- call super setUp in DAPackageDependencyWrapperTest
- call super setUp in DAPackageRelationGraphDiffTest
- call super setUp in DAPackageRelationGraphTest (with some formatting cleanups in setUp)
- call super setUp in DAPackageTest (with some formatting cleanups in setUp)
- call super setUp in DATarjanAlgorithmTest (with some formatting cleanups in setUp)
Also add some tags to to better categorize the test classes in package "Tool-Dependency-Analyse-Test"
@astares
Copy link
Member Author

astares commented Dec 6, 2017

@astares
Copy link
Member Author

astares commented Dec 7, 2017

CI says:

Could not update commit status. Message: Server returned HTTP response code: 201, message: 'Created' for URL: https://api.github.com/repos/pharo-project/pharo/statuses/61ea95635725222feec73e11f117f54794b9454b

ERROR: Out of retries running tests
Finished: FAILURE

@MarcusDenker MarcusDenker merged commit 91c4755 into pharo-project:development Dec 7, 2017
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.

None yet

2 participants