Skip to content

Commit

Permalink
fix failed test
Browse files Browse the repository at this point in the history
  • Loading branch information
dareumnam committed Jul 18, 2023
1 parent 5ea4c2a commit a8a41ed
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/EnergyPlus/PollutionModule.cc
Original file line number Diff line number Diff line change
Expand Up @@ -276,8 +276,8 @@ void GetPollutionFactorInput(EnergyPlusData &state)
state.dataIPShortCut->cAlphaFieldNames,
state.dataIPShortCut->cNumericFieldNames);

FuelType.FuelTypeNames(Loop) = static_cast<Constant::eFuel>(
getEnumerationValue(Constant::eFuelNamesUC, UtilityRoutines::MakeUPPERCase(state.dataIPShortCut->cAlphaArgs(1))));
FuelType.FuelTypeNames(Loop) =
static_cast<Constant::eFuel>(getEnumValue(Constant::eFuelNamesUC, UtilityRoutines::makeUPPER(state.dataIPShortCut->cAlphaArgs(1))));

switch (FuelType.FuelTypeNames(Loop)) {
case Constant::eFuel::NaturalGas: {
Expand Down
1 change: 1 addition & 0 deletions tst/EnergyPlus/unit/PollutionModule.unit.cc
Original file line number Diff line number Diff line change
Expand Up @@ -469,6 +469,7 @@ TEST_F(EnergyPlusFixture, PollutionModule_TestOutputVariables)
ASSERT_TRUE(process_idf(idf_objects));

state->dataPollutionModule->FuelType.FuelTypeNames.allocate(10);
state->dataPollutionModule->FuelType.FuelTypeNames = Constant::eFuel::Invalid;
state->dataPollutionModule->GetInputFlagPollution = true;
PollutionModule::SetupPollutionMeterReporting(*state);

Expand Down

5 comments on commit a8a41ed

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pollutant (dareumnam) - Win64-Windows-10-VisualStudio-16: OK (2667 of 2667 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-3
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pollutant (dareumnam) - x86_64-MacOS-10.17-clang-13.0.0: OK (3405 of 3440 tests passed, 579 test warnings)

Messages:\n

  • 612 tests had: AUD diffs.
  • 562 tests had: MDD diffs.
  • 612 tests had: MTD diffs.
  • 562 tests had: RDD diffs.
  • 14 tests had: Table big diffs.
  • 23 tests had: ESO big diffs.
  • 23 tests had: MTR big diffs.

Failures:\n

regression Test Summary

  • Passed: 737
  • Failed: 35

Build Badge Test Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pollutant (dareumnam) - x86_64-Linux-Ubuntu-22.04-gcc-11.3: OK (3405 of 3481 tests passed, 700 test warnings)

Messages:\n

  • 773 tests had: AUD diffs.
  • 676 tests had: MDD diffs.
  • 773 tests had: MTD diffs.
  • 676 tests had: RDD diffs.
  • 26 tests had: Table big diffs.
  • 56 tests had: ESO big diffs.
  • 56 tests had: MTR big diffs.

Failures:\n

regression Test Summary

  • Passed: 716
  • Failed: 76

Build Badge Test Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pollutant (dareumnam) - x86_64-Linux-Ubuntu-22.04-gcc-11.3-IntegrationCoverage-Debug: OK (776 of 776 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

@nrel-bot-2b
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pollutant (dareumnam) - x86_64-Linux-Ubuntu-22.04-gcc-11.3-UnitTestsCoverage-Debug: OK (1894 of 1894 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.