JetBrains / JetBrains/CMake-Tutorial-sample

assertEquals from assert_macro.h actually is assertTrue

Open
#2 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
40
Forks
13
PR merge metrics
No merged PRs in 30d

Description

assertEquals would need two have two parameters, not just one.

// misnomer:
assertEquals( (0+0) == 0 )
// should be:
assertTrue( (0+0) == 0 )

// assertEquals I would expect to call
// providing both the expected value and the actual value:
assertEquals( 0, (0+0) )

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.