Project

General

Profile

Actions

Feature #411

open

Fix unit tests to run on all supported environments (non-Linux, clang vs gcc, etc)

Added by Anastasia Klimchuk about 2 years ago. Updated over 1 year ago.

Status:
In Progress
Priority:
Normal
Category:
Build system
Target version:
Start date:
09/01/2022
Due date:
% Done:

70%

Estimated time:
Affected versions:
Needs backport to:
Affected hardware:
Affected OS:

Description

The list of issues to fix below (can be updated if new issues found).

1) DONE Mocks for _fprintf_chk are missing (needed for clang on Linux)
2) DONE I/O mock functions need to be renamed so that they do not have name clash with standard I/O, because the latter are allowed to be macros
Current idea is to add iom_ prefix for all functions members of struct io_mock
3) DONE Refactor the code so that linux/spidev.h header is only included when building for Linux
4) DONE Fix failing tests on FreeBSD (4 tests in chip.c: 2 write chip and 2 verify chip)
5) DONE BSD family
6) MacOS needs verifying

Actions #1

Updated by Anastasia Klimchuk about 2 years ago

  • Description updated (diff)
Actions #2

Updated by Felix Singer about 2 years ago

What's the status here? Any progress? Is it resolved?

Actions #3

Updated by Anastasia Klimchuk about 2 years ago

  • % Done changed from 0 to 50

Oh yes there is progress! 3 and 4 done, and those were the challenging part. So I am marking 50% done.

1 and 2 left, which needs more code than 3 and 4, but that is way more straightforward task.

Actions #4

Updated by Anastasia Klimchuk almost 2 years ago

1) and 2) done as well.

Actions #5

Updated by Anastasia Klimchuk almost 2 years ago

  • Description updated (diff)
  • % Done changed from 50 to 70

I added 5) to the description

Actions #6

Updated by Anastasia Klimchuk almost 2 years ago

  • Parent task deleted (#353)
Actions #7

Updated by Anastasia Klimchuk almost 2 years ago

  • Category changed from Release prep to Build system
Actions #8

Updated by Anastasia Klimchuk over 1 year ago

  • Description updated (diff)
Actions

Also available in: Atom PDF