Skip to content
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

Added google test and moved tests. #577

Merged
merged 62 commits into from
Jul 31, 2024
Merged

Added google test and moved tests. #577

merged 62 commits into from
Jul 31, 2024

Conversation

ChristianFeldmann
Copy link
Member

@ChristianFeldmann ChristianFeldmann commented Jul 7, 2024 •
edited
Loading

  • Get googletest as a submodule and compile it with qmake.
  • Ported all current tests to Googletest.
  • Refactor EnumMapper to be constexpr

@ChristianFeldmann
Added google test and moved a first test over (file name guessing test).
0f4e2bb
@ChristianFeldmann
Checkout submodules
a420b9f
@ChristianFeldmann
Remove old unit test files
38b3269
@ChristianFeldmann
Add new implementation of FileSourceAnnexB test
7e9371f
@ChristianFeldmann
Port test for statistics data
ce1650f
@ChristianFeldmann
Add StatisticsFileCSVTest
9b9a65d
@ChristianFeldmann
Add VTBMS stats test
e9bd096
@ChristianFeldmann
Add new enum mapper and first conversion to rgb test
42ea197
@ChristianFeldmann
Use string view in new mapper
4052cec
@ChristianFeldmann
Add some more mappers
9a09483
@ChristianFeldmann
Convert another enum mapper
e0bc883
@ChristianFeldmann
Convert InputFormatMapper
ec3f75f
@ChristianFeldmann
Convert NalTypeMapper
64d7a9e
@ChristianFeldmann
Converted HEVC NalTypeMapper
1998ee9
@ChristianFeldmann
Use other initialization
7025047
@ChristianFeldmann
Adapt APSParamTypeMapper
b95d402
@ChristianFeldmann
Switch NalTypeMapper and add SliceTypeMapper for VVC
f956279
@ChristianFeldmann
Switch OverlayLayoutModeMapper
88e850d
@ChristianFeldmann
Switch all enums in the ColorMapper
2b6e62b
@ChristianFeldmann
Convert RepeatMode
f051e1e
@ChristianFeldmann
Switch DataLayoutMapper
11982c2
@ChristianFeldmann
Switch ChannelOrderMapper
a3c42bb
@ChristianFeldmann
Switch ComponentDisplayMode. I think we can get rid of the text part …
64aeb0c 
…in the enum mapper.
@ChristianFeldmann
Switch ComponentDisplayModeMapper
79a01ea
@ChristianFeldmann
Switch ColorConversionMapper
c4f50ac
@ChristianFeldmann
Switch ChromaInterpolationMapper and PredefinedPixelFormatMapper
e94d7e4
@ChristianFeldmann
Switch PackingOrderMapper
efb956f
@ChristianFeldmann
Switch PlaneOrderMapper
6cc2c0e
@ChristianFeldmann
Convert SubsamplingMapper
854ca3e
@ChristianFeldmann
Remove include. That should be the last one.
498d455
@ChristianFeldmann
Add RGB conversions and getPixel value tests
62ebe85
@ChristianFeldmann
Port issue test
50c04f4
@ChristianFeldmann
Add pixelFormatRGB test
5dad47d
@ChristianFeldmann
Improve test case naming function
3da4a31
@ChristianFeldmann
Add PixelFormatRGBTest
a0c7fd5
@ChristianFeldmann
Started adding PixelFormatYUV tests
6151744
@ChristianFeldmann
Add remaining tests
1841bee
@ChristianFeldmann
Add pixel format tests
2f9f75c
@ChristianFeldmann
Test only windows build
fc05bfd
@ChristianFeldmann
Check directories for lib
943ffcb
@ChristianFeldmann
Updated path
a906cde
@ChristianFeldmann
List debug and release directories
8d6c98b
@ChristianFeldmann
Add test for EnumMapper
843b6cc
@ChristianFeldmann
Test qmake flags
bc3d504
@ChristianFeldmann
Also update qmock pro file
75517d4
@ChristianFeldmann
Compilation fix
2d98b70
@ChristianFeldmann
Test running of tests
871680f
@ChristianFeldmann
Try running unit test on mac/linux
c910a9f
@ChristianFeldmann
Merge branch 'develop' into feature/addGoogleTestAsSubmodule
36bdafe
@ChristianFeldmann
Yaml syntax fix
a755bfd
@ChristianFeldmann
Checkout submodules on windows and unix native
6f3427e
@ChristianFeldmann
Fix app packaging macos14. Also test native builds.
d831340
@ChristianFeldmann
Add submodules on mac native
e78fbec
@ChristianFeldmann ChristianFeldmann marked this pull request as ready for review July 29, 2024 16:09
ChristianFeldmann
ChristianFeldmann commented Jul 31, 2024
View reviewed changes
Copy link
Member Author

@ChristianFeldmann ChristianFeldmann left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. To-do:

  • Rename EnumMapper getItems. Should be getValues.
  • Test running of YUView

@ChristianFeldmann ChristianFeldmann changed the title Added google test and moved a first test over (file name guessing test). Added google test and moved tests. Jul 31, 2024
@ChristianFeldmann
Copy link
Member Author

Check and check.

@ChristianFeldmann ChristianFeldmann merged commit 8e13a10 into develop Jul 31, 2024
9 checks passed
@ChristianFeldmann ChristianFeldmann deleted the feature/addGoogleTestAsSubmodule branch July 31, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ChristianFeldmann