Support Test Impact Analysis #154

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

Draft

mpdeimos wants to merge 199 commits into master from tia

Contributor

mpdeimos commented Feb 2, 2021 •

edited by Raphael-N

Loading

Addresses issue TS-38029

Tests written
Documentation updated
CHANGELOG.md updated
Present new features in N&N

Please respect the Teamscale vote. Clean up findings before sending your change into review. Blacklist and report false-positives. Adjust the analysis profile if needed.

mpdeimos and others added 30 commits

September 28, 2020 14:57


          zmq powered gui

065c418


          retarget vc projects

2385d65


          codemaid

a1e3e9f


          add ZMQ, first concept of one-way profiler communication

e0d81b1


          add testcase for one-way profiler communication

a64c664


          rename gui

f0c0be3


          add profiler commons project and use in test

5befda8


          Test infra: use .net ZMQ and native ZMQ

95d8074


          Extend Profiler Testee with an interactive mode

59af32f


          Add publisher socket to C# IPC implementation

2f6f80d


          Wire subscribe socket in C++ IPC

5c63ba7


          write function infos to log on test case switch

d40729c


          Real TIA method call profiling

1d016b9


          Refactor tests

ea2fa00


          No locking until IPC is started

2f2e923


          Use NetMQ-native threading


          compiler profiler testee with native bitness and as x86, x64

a5d3519


          parameterize tia tests as x64 and x86

aee3e91


          test start ipc after app (not working yet)

1fbf7e7


          commander rework

82ea47f


          simplified commander

3874d31


          multipart tia start/end

519a256


          escaping of characters

b421dc6


          store test execution result

0fec463


          disable mscorlib profiling for TIA

0bf704e


          fix for default socket

8d868f8


          Wrap coverage data in report class.

b256b52


          Write testwise coverage report.

efa1a83


          Merge TraceFile and ParsedTraceFile classes.

6f639cf


          Hide internal data of TraceFile.

266c6c0

wilhelma reviewed

View reviewed changes

Profiler/CProfilerCallback.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/log/TraceLog.h Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/log/TraceLog.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/log/TraceLog.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/log/TraceLog.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/log/TraceLog.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/log/TraceLog.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.cpp Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.h Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/Ipc.h Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/MethodEnter.h Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/MethodEnter.h Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/functionID_set/functionId_set.h Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/functionID_set/functionId_set.h Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/functionID_set/functionId_set.h Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/functionID_set/functionId_set.h Outdated Show resolved Hide resolved

wilhelma reviewed

View reviewed changes

Profiler/utils/functionID_set/functionId_set.h Outdated

    
              	/// <summary>

              	/// True if the set contains FunctionID f, false otherwise.

              	/// </summary>

              	bool contains(FunctionID f) {

wilhelma Mar 27, 2025

I don't understand the logic... Is this really so complicated or can it be used by an algorithm from <algorithm>?

Raphael-N requested a review from wilhelma

July 23, 2025 12:22


          Rework

2861c06

Raphael-N force-pushed the tia branch from 49b8920 to 2861c06 Compare

July 30, 2025 08:43

wilhelma approved these changes

View reviewed changes

wilhelma left a comment

Changes look good. Is the behavior identical?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet