-
Notifications
You must be signed in to change notification settings - Fork 217
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
Enhancements to OSAL UT stub code in osapi_stubs #93
Comments
Imported from trac issue 70. Created by jphickey on 2015-07-02T13:31:38, last modified: 2015-11-20T16:22:16 |
Trac comment by jphickey on 2015-07-02 14:43:52: Pushed commit [changeset:1f5c968] for review. |
Trac comment by jphickey on 2015-07-09 14:52:53: Note - the above commit is merged into the {{{ic-2015-07-14}}} integration candidate branch for testing. Since the rest of the unit testing framework is already included, it makes sense that this be merged in as well. |
Trac comment by jphickey on 2015-07-10 12:53:02: Additional commit pushed which adds more stubs for timebase API functions {{{OS_TimeBaseCreate}}}, {{{OS_TimeBaseDelete}}}, etc. Commit [changeset:3bb3ee4] Also added to the {{{ic-2015-07-14}}} merge |
Trac comment by acudmore on 2015-07-10 15:05:47: recommend accept |
Trac comment by glimes on 2015-07-13 14:13:41: Accept. It looks like this was rebased and changeset [changeset:1f5c968] is actually |
Trac comment by jphickey on 2015-07-13 14:17:04: Replying to [comment:6 glimes]:
Actually the ones in [changeset:1f5c968] are a different set of stubs. There are two commits adding a different set of stubs, I kept them separate just in case anyone had already looked at the first diff. After merging both of these change sets we should have a stub for every currently defined OSAL function. |
Trac comment by glimes on 2015-07-13 19:17:11: Ah, I see it now. Slight tool reorg was needed ;) Yep, Accept [changeset:3bb3ee4]. |
Trac comment by sduran on 2015-07-14 12:21:19: accept |
Trac comment by jwilmot on 2015-07-14 12:36:10: Recommend accept. |
Trac comment by glimes on 2015-07-16 10:35:28: Included in [changeset:ec67bd4 2015-07-14 development merge], |
This ticket reflects further testing with CFE unit tests. Previous ticket #40 is now integrated with CFE unit tests but some additional bits are needed to make everything right. Since #40 was approved at the last CCB, it is not going to be amended anymore, so this is a follow-on update.
The text was updated successfully, but these errors were encountered: