Carrot2

Empty metadata values.

Details

  • Type: Sub-task Sub-task
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 3.4.0
  • Component/s: Framework Core
  • Labels:
    None

Description

Staszek, peek at testEmptyTitle() in BindableMetadataBuilderTest – it fails with the new code (the only test failing) because empty fields are set to null, not an empty string. Should it be an empty string instead, or is this test invalid?

Activity

Hide
Stanisław Osiński added a comment -

The test is valid in a sense that it checks a valid input (empty title). But the result can be either null or "", does not matter. The code up the stack should be able to handle both results.

Show
Stanisław Osiński added a comment - The test is valid in a sense that it checks a valid input (empty title). But the result can be either null or "", does not matter. The code up the stack should be able to handle both results.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: