Skip to content

Initial unit tests for cleardep#1

Open
rmst wants to merge 9 commits intosugar-lang:masterfrom
rmst:master
Open

Initial unit tests for cleardep#1
rmst wants to merge 9 commits intosugar-lang:masterfrom
rmst:master

Conversation

@rmst
Copy link
Copy Markdown

@rmst rmst commented Sep 2, 2014

No extra features or bugfixes jet

@seba--
Copy link
Copy Markdown
Member

seba-- commented Sep 3, 2014

Please resolve the following two issues first.

  1. Can you rename TestCompilationUnit to CompilationUnitMock?
  2. The new source folder test should be registered in some project configuration file, right? Please also commit that file. Futhermore, build.properties lists source folders and probably needs adaption.

2. corresponding unit test added
3. addCirculrModuleDependency(..) depreciated
-corrected visitor-tests
-minor other changes
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.

2 participants