crp/OpenMC
1
0
Fork 0
mirror of https://github.com/openmc-dev/openmc.git synced 2026-07-26 13:15:39 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Adding test for name setting and relying on std::string's char* constructor.

Browse source
This commit is contained in:
Patrick Shriwise 2019-07-19 10:03:22 -05:00
parent f2199af0eb
commit 331e198617
3 changed files with 5 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

3
src/cell.cpp
View file

@ -1102,8 +1102,7 @@ openmc_cell_set_name(int32_t index, const char* name) {
return OPENMC_E_OUT_OF_BOUNDS;
}
std::string name_str(name);
model::cells[index]->set_name(name_str);
model::cells[index]->set_name(name);
return 0;
}

1
src/material.cpp
View file

@ -1400,7 +1400,6 @@ openmc_material_set_name(int32_t index, const char* name) {
return OPENMC_E_OUT_OF_BOUNDS;
}
std::string name_str(name);
model::materials[index]->set_name(name);
return 0;

4
tests/unit_tests/test_capi.py
View file

@ -75,6 +75,8 @@ def test_cell(capi_init):
cell.fill = openmc.capi.materials[1]
assert str(cell) == 'Cell[0]'
assert cell.name == "Fuel"
cell.name = "Not fuel"
assert cell.name == "Not fuel"
def test_cell_temperature(capi_init):
cell = openmc.capi.cells[1]
@ -124,6 +126,8 @@ def test_material(capi_init):
m.set_density(0.1, 'g/cm3')
assert m.density == pytest.approx(0.1)
assert m.name == "Hot borated water"
m.name = "Not hot borated water"
assert m.name == "Not hot borated water"
def test_material_add_nuclide(capi_init):
m = openmc.capi.materials[3]