Skip to content
This repository has been archived by the owner on Sep 24, 2020. It is now read-only.

[Tests] Add missing core Tests #170

Open
4 of 6 tasks
ddmler opened this issue Oct 4, 2016 · 1 comment
Open
4 of 6 tasks

[Tests] Add missing core Tests #170

ddmler opened this issue Oct 4, 2016 · 1 comment
Labels
good first issue help wanted tests

Comments

@ddmler
Copy link
Collaborator

ddmler commented Oct 4, 2016 •
edited

src directory:

  • AliasManager
  • Application
  • Compiler
  • Variable
  • ScopePointer
  • IssuesCollector

are completely missing. Context needs some more tests.
(Configuration but this is changed a lot in a PR so we will wait for merge there)
@DannyvdSluijs
Copy link

Will look into this :-)

This was referenced Oct 4, 2016
Closed
Merged
Open
DannyvdSluijs pushed a commit to DannyvdSluijs/phpsa that referenced this issue Oct 5, 2016
Add core unit tests (ovr#170)
7fe88e7
@DannyvdSluijs DannyvdSluijs mentioned this issue Oct 5, 2016
Merged
8 tasks
ddmler pushed a commit that referenced this issue Nov 21, 2016
Merge pull request #183 from DannyvdSluijs/master
35ec269 
Add core unit tests (#170)
ovr pushed a commit that referenced this issue Nov 22, 2016
@ovr
Add core unit tests (#170)
f80883f
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
good first issue help wanted tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ddmler @DannyvdSluijs