cms-bot icon indicating copy to clipboard operation
cms-bot copied to clipboard

Unit tests for process_pr script

Open iarspider opened this issue 1 year ago • 11 comments

Replaces #2193. I will squash all commits once the PR is ready to be merged.

iarspider avatar Apr 16 '24 14:04 iarspider

A new Pull Request was created by @iarspider for branch master.

@iarspider, @cmsbuild, @aandvalenzuela, @smuzaffar can you please review it and eventually sign? Thanks. @rappoccio, @antoniovilela, @sextonkennedy you are the release manager for this. cms-bot commands are listed here

cmsbuild avatar Apr 16 '24 14:04 cmsbuild

cms-bot internal usage

cmsbuild avatar Apr 16 '24 14:04 cmsbuild

Pull request #2213 was updated.

cmsbuild avatar Apr 16 '24 14:04 cmsbuild

@smuzaffar @aandvalenzuela the core of this PR is ready, just need to prepare data for one more test (detection of backdated commit) and write github action to run this test suite.

iarspider avatar Apr 17 '24 08:04 iarspider

The tests are somewhat fragile: if the mapping between file and category, or category and watchers is changed, the tests will fail, since they expect the exact comment text to match.

iarspider avatar Apr 17 '24 14:04 iarspider

Pull request #2213 was updated.

cmsbuild avatar Apr 18 '24 12:04 cmsbuild

Pull request #2213 was updated.

cmsbuild avatar Apr 30 '24 14:04 cmsbuild

Pull request #2213 was updated.

cmsbuild avatar May 03 '24 11:05 cmsbuild

Pull request #2213 was updated.

cmsbuild avatar May 03 '24 12:05 cmsbuild

Pull request #2213 was updated.

cmsbuild avatar May 03 '24 12:05 cmsbuild

Pull request #2213 was updated.

cmsbuild avatar May 03 '24 12:05 cmsbuild

Pull request #2213 was updated.

cmsbuild avatar May 17 '24 12:05 cmsbuild

Pull request #2213 was updated.

cmsbuild avatar May 17 '24 12:05 cmsbuild