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.

Sign up for GitHub

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

Jump to bottom

Write tests for initial value initial value specification D.8 #175

Closed
3 tasks
killenb opened this issue Jul 29, 2020 · 1 comment
Closed
3 tasks

Write tests for initial value initial value specification D.8 #175

killenb opened this issue Jul 29, 2020 · 1 comment
Assignees
@jhktimm

Comments

@killenb
Copy link
Member

killenb commented Jul 29, 2020 •
edited
Loading

Currently not all implementations behave according to the spec, but all tests are passing. Hence essential tests are missing.
Also internal modules must implement this.

At least the following places must be checked with push-type and poll-type variables:

  • ApplicationModule
  • TriggerFanOut
  • FeedingFanOut

The implementation is done in #176

This ticket is part of #179, the ticket to write systematic tests. When writing tests name and link them to the specification as mentioned there.
@killenb killenb mentioned this issue Jul 29, 2020
Closed
3 tasks
@killenb killenb assigned killenb and unassigned killenb Jul 29, 2020
@jhktimm jhktimm assigned jhktimm and unassigned jhktimm Jul 29, 2020
@killenb killenb mentioned this issue Aug 5, 2020
Open
2 tasks
@killenb killenb mentioned this issue Aug 5, 2020
Closed
@jhktimm jhktimm self-assigned this Aug 5, 2020
jhktimm added a commit that referenced this issue Aug 12, 2020
@jhktimm
tests for initial value D.8 #175
67dd219 
	Write tests for initial value initial value specification D.8
@jhktimm jhktimm added the review label Aug 13, 2020
@killenb
Copy link
Member Author

killenb commented Aug 28, 2020

Push-type tests for ProcessArray and constants was missing. Added them.

@killenb killenb closed this as completed Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jhktimm jhktimm

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jhktimm @killenb