Add some unit tests for KDevMI::MI's MIParser, MiCommands, CommandQueue
ClosedPublic

Authored by kossebau on Jul 22 2018, 3:01 PM.

Diff Detail

Repository
R32 KDevelop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau created this revision.Jul 22 2018, 3:01 PM
Restricted Application added a project: KDevelop. · View Herald TranscriptJul 22 2018, 3:01 PM
Restricted Application added a subscriber: kdevelop-devel. · View Herald Transcript
kossebau requested review of this revision.Jul 22 2018, 3:01 PM

Created while trying to understand the MI codebase and to make sure the code/API watched at works. No issues yet flound in the actual code.
Possibly more to come in the next weeks.

apol added a subscriber: apol.Jul 22 2018, 10:44 PM

+1
Looks sound to me, @peifengyu may know more.

apol added a comment.Jul 22 2018, 10:45 PM

I guess peifengyu is not on phab :(

arrowd added a subscriber: arrowd.Jul 25 2018, 6:29 AM

Just a note that these tests pass on FreeBSD.

With no code expert around and no objections, guess will simply push then upcoming WE (Aug 4/5).

This revision was not accepted when it landed; it landed in state Needs Review.Aug 4 2018, 4:36 PM
This revision was automatically updated to reflect the committed changes.