This change has been marked as completed.

Describe the completed change (optional):

Summary

Add Birb.assert_list_mode_n_items

Review Request #3936 — Created March 28, 2025 and submitted 5 days, 14 hours ago

Information

Owner

grim*

Repository

birb/birb

Branch

default

Bugs

BIRB-15

Depends On

~~3935~~

Reviewers

Groups

pidgin

People

Description*

This is a helper to shorten the amount of code we have to type when we do this
multiple times in a unit test.

Testing Done

Ran the count tests under valgrind and called in the turtles.

Commits

Summary	ID
Add Birb.assert_list_mode_n_items This is a helper to shorten the amount of code we have to type when we do this multiple times in a unit test.	cfc6dba596138c370a10f591be9dc763f7eab28e

Issues

Description	From	Last Updated
The downside of this is that the file/line are incorrect for the assertion. It would have to be a macro …	QuLogic	5 days, 14 hours ago
Also missing a BIRB_AVAILABLE_ tag.	QuLogic	5 days, 14 hours ago
There are no open issues

birb/birbassert.h (Diff revision 1)

The issue has been resolved. Show all issues

The downside of this is that the file/line are incorrect for the assertion. It would have to be a macro to match g_assert_cmpuint, which would get the right file/line.

birb/birbassert.h (Diff revision 1)
The issue has been resolved. Show all issues
```
Also missing a BIRB_AVAILABLE_ tag.
```

Change Summary:

move to a macro and eliminate the need for a cast.

Commits:

	Summary	ID
	Add Birb.assert_list_mode_n_items This is a helper to shorten the amount of code we have to type when we do this multiple times in a unit test.	9117c506a7db947b6d0e8bd1daccea54beaf16fc
	Add Birb.assert_list_mode_n_items This is a helper to shorten the amount of code we have to type when we do this multiple times in a unit test.	cfc6dba596138c370a10f591be9dc763f7eab28e

Diff:

Revision 2 (+40)

Show changes

	birb/birbassert.h
	birb/tests/test_count.c

Ship it!

```
Ship It!
```

Status:: Completed