Skip to content

Add mocks#80

Closed
Lana243 wants to merge 25 commits intomainfrom
Lana243/mocks
Closed

Add mocks#80
Lana243 wants to merge 25 commits intomainfrom
Lana243/mocks

Conversation

@Lana243
Copy link
Collaborator

@Lana243 Lana243 commented Apr 12, 2023

Summary:

Checklist:

  • The PR addresses a single issue. If it can be divided into multiple independent PRs, please do so.
  • The PR is divided into a logical sequence of commits OR a single commit is sufficient.
  • There are no unnecessary commits (e.g. commits fixing issues in a previous commit in the same PR).
  • Each commit has a meaningful message documenting what it does.
  • All messages added to the codebase, all comments, as well as commit messages are spellchecked.
  • The code is commented OR not applicable/necessary.
  • The patch is formatted via clang-format OR not applicable (if explicitly overridden leave unchecked and explain).
  • There are test cases for the code you added or modified OR no such test cases are required.

@Lana243 Lana243 changed the title Add naive mocks and replay for those Mocks Apr 12, 2023
@Lana243 Lana243 changed the title Mocks Add mocks Apr 12, 2023
@Lana243 Lana243 force-pushed the Lana243/mocks branch 2 times, most recently from bda9b8e to 98f876d Compare April 24, 2023 13:37
@Lana243 Lana243 force-pushed the Lana243/mocks branch 4 times, most recently from c5547e5 to 0a82cb8 Compare June 19, 2023 15:02
@Lana243 Lana243 marked this pull request as ready for review June 22, 2023 15:49
@misonijnik misonijnik force-pushed the Lana243/mocks branch 2 times, most recently from f818012 to baec6f6 Compare June 28, 2023 19:44
@codecov-commenter
Copy link

codecov-commenter commented Jul 24, 2023

Codecov Report

Merging #80 (0f1640e) into main (e9dc5f3) will increase coverage by 0.00%.
The diff coverage is 64.86%.

❗ Current head 0f1640e differs from pull request most recent head 6f91181. Consider uploading reports for the commit 6f91181 to get more accurate results

@@           Coverage Diff            @@
##             main      #80    +/-   ##
========================================
  Coverage   65.12%   65.13%            
========================================
  Files         209      211     +2     
  Lines       28462    28861   +399     
  Branches     6335     6424    +89     
========================================
+ Hits        18537    18798   +261     
- Misses       7425     7527   +102     
- Partials     2500     2536    +36     
Files Changed Coverage Δ
include/klee/Expr/IndependentSet.h 56.66% <ø> (ø)
lib/Core/ExecutionState.cpp 73.91% <ø> (ø)
lib/Core/Executor.h 76.92% <ø> (ø)
lib/Core/SpecialFunctionHandler.h 0.00% <ø> (ø)
lib/Expr/Parser.cpp 55.11% <0.00%> (-1.66%) ⬇️
runtime/Mocks/models.c 0.00% <0.00%> (ø)
lib/Core/SpecialFunctionHandler.cpp 72.48% <30.00%> (-0.98%) ⬇️
lib/Expr/ExprPPrinter.cpp 88.56% <30.00%> (-1.47%) ⬇️
tools/klee/main.cpp 58.20% <51.85%> (-0.93%) ⬇️
lib/Expr/IndependentSet.cpp 77.11% <63.63%> (-0.90%) ⬇️
... and 10 more

... and 2 files with indirect coverage changes

@misonijnik misonijnik force-pushed the Lana243/mocks branch 5 times, most recently from f02424e to 4be9c8d Compare July 27, 2023 15:13
@misonijnik
Copy link
Collaborator

#109 PR incorporates this PR but also advances further.

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.

3 participants