Ticket #7366 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

In IDFs check all from dates later than 1900-01-31 23:59:01

Reported by: Anders Markvardsen Owned by: Anders Markvardsen
Priority: critical Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Keith Brown

Description

Accidently found that if a data file load has a data matching a IDF with a from data prior to 1900-01-31 23:59:01 then the data file loads but not the IDF

In this ticket ensure that no from dates are earlier than 1900-01-31 23:59:01, which at least EMU_Definition_32detectors.xml has

Change History

comment:1 Changed 7 years ago by Anders Markvardsen

A couple with invalid valid-from dates. re #7366

Changeset: 49310769ec8c424378d4e92ac139e9775564b942

comment:2 Changed 7 years ago by Anders Markvardsen

Throw if valid-from not valid. re #7366

To help user and developer print a sensible message is valid-from date is too early

Changeset: a3ab9a78b6936a826a1a0fb4a0e2d6d6de2f9270

comment:3 Changed 7 years ago by Russell Taylor

Note the warning (on all platforms I think) due to there being only one colon after 'Kernel' at Instrument.cpp line 1126:

Kernel:DateAndTime earliestAllowedDate("1900-01-31 23:59:01");

comment:4 Changed 7 years ago by Anders Markvardsen

fix warning russell spotted. re #7366

Changeset: 98605c7e5cbd292d359191b92fa26165fdc92c90

comment:5 Changed 7 years ago by Anders Markvardsen

  • Status changed from new to accepted

To test check that unit tests and systemtests are passing

In instrument folder check that there are no IDF with a valid-from date earlier than 1900-01-31 23:59:01

comment:6 Changed 7 years ago by Anders Markvardsen

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:7 Changed 7 years ago by Keith Brown

  • Status changed from verify to verifying
  • Tester set to Keith Brown

comment:8 Changed 7 years ago by Keith Brown

  • Status changed from verifying to closed

Tested that it throws with an invalid date having altered a valid file to become invalid then tried loading it. Reverted that edit and loaded it again having fixed the date back to how it was

LoadLotsOfInstruments test passed during the last build on is_test_rhel6_develop which was ran after the last commit for this branch so all instruments pass this test.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8212

Note: See TracTickets for help on using tickets.