Add unit testing framework. #21

SharafMohamed · 2024-08-07T01:15:09Z

Description

Add unit test framework.
Add a unit test for creating a schema object.
Add install script for Catch2.
Add Catch2 and unit tests to workflow.
Add unit tests to linter.
Update README to include Catch2.

Validation performed

Unit tests compile and run successfully.
Unit tests run successfully in workflow.

…es are; Add '<','>','?' to special_regex_characters for capture groups, and add productions to cancel their literals; Add CaptureAST

…tion

…tead of value insertion

… copy constructor; Explicitly specify variable names in RegexASTCapture constructor

Co-authored-by: Lin Zhihao <[email protected]>

submodules/Catch2

tests/test-lexer.cpp

tests/CMakeLists.txt

CMakeLists.txt

tests/test-lexer.cpp

…nstructions.

tests/CMakeLists.txt

CMakeLists.txt

.github/workflows/build.yaml

Co-authored-by: kirkrodrigues <[email protected]>

…Add sudo to install lines; Change |- to |

…abetize

kirkrodrigues

For the PR title, how about:

Add unit testing framework.

SharafMohamed and others added 15 commits July 26, 2024 06:45

Add special_regex_characters to cpp so its clear what all lexical rul…

bc1c411

…es are; Add '<','>','?' to special_regex_characters for capture groups, and add productions to cancel their literals; Add CaptureAST

Handle capture groups in schema file, but not yet handled in NFA crea…

00b7e7b

…tion

Merge remote-tracking branch 'upstream/main' into tagged-dfa

b8f9ab6

Make SchemaParser::m_special_regex_characters inline; Use emplace ins…

0597138

…tead of value insertion

Copy m_name in RegexASTCapture copy constructor.

f35bb68

Rename m_name to m_group_name in RegexASTCapture; Add m_group_name to…

dcc21c4

… copy constructor; Explicitly specify variable names in RegexASTCapture constructor

Move RegexASTCapture constructor into header

f17ccef

Move group_name when constructing RegexASTCapture

e0a6689

Fix header comment

77bb069

Fix comments in RegexASTCapture class.

cf9d70b

Change for constant == variable

bb43cd2

Rename m_regex to m_group_regex_ast; include <string>

aa0c32a

Update src/log_surgeon/SchemaParser.cpp

96aa2b0

Co-authored-by: Lin Zhihao <[email protected]>

Initial framework added for unittest

ad43334

Add tests to linter

dc064d4

SharafMohamed changed the title ~~Add unit test framework~~ Add unit test framework. Aug 7, 2024

SharafMohamed added 9 commits August 6, 2024 21:18

Add submoudles to workflow

ff83d1c

Run unitTest in workflow

5176c63

Run unitTest in workflow, again

bf63701

Run unitTest in workflow, again again

fa3e512

Test for unitTest in workflow fix

0e9678f

Test for unitTest in workflow fix 2

33d45c8

Test for unitTest in workflow fix 3

9a82514

Remove duplicated changes to file from PR#20

f0f120d

Remove 1 more duplicated change from PR#20

e24c3b7

kirkrodrigues requested changes Aug 8, 2024

View reviewed changes

kirkrodrigues reviewed Aug 8, 2024

View reviewed changes

tests/test-lexer.cpp Outdated Show resolved Hide resolved

SharafMohamed added 3 commits August 13, 2024 06:11

Update catch2 version to 4e8d92bf02f7d1c8006a0e7a5ecabd8e62d98502

51b50b4

Update Catch2 to v3

d222bbc

clang-tidy fixes for test-lexer.cpp

d17969c

SharafMohamed and others added 15 commits August 14, 2024 04:15

Catch2 workflow fix 2

9a4695c

Remove dpkg from catch2 install script

35fe9eb

Fix Catch2 for macOS in workflow

43fedcb

Fix Catch2 for Ubuntu in workflow

9cc97dc

Fix Catch2 install steps in workflow

6a7a670

Fix if syntax in workflow

2dc60c6

Add quotations to workflow

d85a6c0

Update .gitmodules to remove Catch2 submodule

927a95a

Remove submodule Catch2 and clean up configuration

3bbf8ac

Update consistency for if b/w build and lint yml files

9c5383a

Reformat catch install script.

449f419

Move catch install script into ubuntu directory.

4a5c610

Add catch2 install instructions.

dc8f395

Fix catch2 install script permissions.

e43e0b2

Update build command in anticipation of review changes; Add testing i…

dfd790d

…nstructions.

kirkrodrigues requested changes Aug 14, 2024

View reviewed changes

SharafMohamed and others added 11 commits August 14, 2024 05:55

Update .github/workflows/build.yaml

e43e1ad

Co-authored-by: kirkrodrigues <[email protected]>

Remove recursive as we no longer have build depending on submodules; …

d7d5640

…Add sudo to install lines; Change |- to |

Add if gaurds to CMakeLists

3c021e4

Change | to |-

873c4ff

Add submodule back to linter

f562747

Use -DBUILD_TESTING=ON and ctest in workflow

125f715

Switch back to install script; Avoid changing working directory; Alph…

3085555

…abetize

Add CMakeLists debugging

c47f0f5

Remove if gaurds from includes needed by examples

476d8ac

Minor refactoring.

985cf57

Add install scripts to workflow dependencies.

6b0c15d

kirkrodrigues approved these changes Aug 14, 2024

View reviewed changes

SharafMohamed changed the title ~~Add unit test framework.~~ Add unit testing framework. Aug 14, 2024

SharafMohamed merged commit d9a686e into y-scope:main Aug 14, 2024
6 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add unit testing framework. #21

Add unit testing framework. #21

SharafMohamed commented Aug 7, 2024 •

edited

Loading

kirkrodrigues left a comment

Add unit testing framework. #21

Add unit testing framework. #21

Conversation

SharafMohamed commented Aug 7, 2024 • edited Loading

Description

Validation performed

kirkrodrigues left a comment

Choose a reason for hiding this comment

SharafMohamed commented Aug 7, 2024 •

edited

Loading