tpunitpp icon indicating copy to clipboard operation
tpunitpp copied to clipboard

expected values

Open kallisti5 opened this issue 8 years ago • 2 comments

It would be nice if the assertions showed expected values instead of just a failure:

#include <NetworkAddress.h>

#include "tpunit++.hpp"


struct BNetworkAddressFixture
    : public tpunit::TestFixture
{
    BNetworkAddressFixture() : tpunit::TestFixture(
        TEST(BNetworkAddressFixture::tSetToIPv4),
        TEST(BNetworkAddressFixture::tSetToIPv6)
    )
    {}

    void tSetToIPv4()
    {
        BNetworkAddress address;
        ASSERT_TRUE(address.IsEmpty());
        address.SetTo("127.0.0.1");
        ASSERT_FALSE(address.IsEmpty());
        ASSERT_TRUE(address.Family() == AF_INET);
    }

    void tSetToIPv6()
    {
        BNetworkAddress address;
        ASSERT_TRUE(address.IsEmpty());
        address.SetTo("::1");
        ASSERT_FALSE(address.IsEmpty());
        ASSERT_TRUE(address.Family() == AF_INET6);
    }
} __BNetworkAddressFixture;
/Builds/bunittest> ./kits/network/BNetworkAddress 
[--------------]
[ RUN          ] BNetworkAddressFixture::tSetToIPv4
[       PASSED ] BNetworkAddressFixture::tSetToIPv4
[ RUN          ] BNetworkAddressFixture::tSetToIPv6
[              ]    assertion #1 at kits/network/BNetworkAddress.cpp:29
[       FAILED ] BNetworkAddressFixture::tSetToIPv6
[--------------]

[==============]
[ TEST RESULTS ] Passed: 1, Failed: 1
[==============]

kallisti5 avatar Jul 26 '15 18:07 kallisti5

Thanks for the feature request. I agree this would make the assertions much more useful. To make this work generically I will need to switch from using printf to std::ostream to make outputting arbitrary types possible.

tpounds avatar Jul 26 '15 21:07 tpounds

This would be just amazing.

iwiznia avatar Aug 09 '18 14:08 iwiznia