kindred/create
1
1
Fork 0
Code Issues 37 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

Spreadsheet: Add tests for rename property

Browse Source
This commit is contained in:
Pieter Hijma
2025-05-18 15:39:53 +02:00
committed by Yorik van Havre
parent 3dde0c0480
commit f114ad7afa
3 changed files with 81 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/src/App/CMakeLists.txt
View File

@@ -17,7 +17,7 @@ target_sources(Tests_run PRIVATE
MappedName.cpp
Metadata.cpp
ProjectFile.cpp
Property.h
Property.h
Property.cpp
PropertyExpressionEngine.cpp
StringHasher.cpp

1
tests/src/Mod/Spreadsheet/App/CMakeLists.txt
View File

@@ -1,5 +1,6 @@
target_sources(Spreadsheet_tests_run PRIVATE
PropertySheet.cpp
RenameProperty.cpp
)
target_include_directories(Spreadsheet_tests_run PUBLIC

79
tests/src/Mod/Spreadsheet/App/RenameProperty.cpp Normal file
View File

@@ -0,0 +1,79 @@
/****************************************************************************
* Copyright (c) 2025 Pieter Hijma <info@pieterhijma.net> *
* *
* This file is part of the FreeCAD CAx development system. *
* *
* This library is free software; you can redistribute it and/or *
* modify it under the terms of the GNU Library General Public *
* License as published by the Free Software Foundation; either *
* version 2 of the License, or (at your option) any later version. *
* *
* This library is distributed in the hope that it will be useful, *
* but WITHOUT ANY WARRANTY; without even the implied warranty of *
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the *
* GNU Library General Public License for more details. *
* *
* You should have received a copy of the GNU Library General Public *
* License along with this library; see the file COPYING.LIB. If not, *
* write to the Free Software Foundation, Inc., 59 Temple Place, *
* Suite 330, Boston, MA 02111-1307, USA *
* *
****************************************************************************/
#include <gtest/gtest.h>
#include <Mod/Spreadsheet/App/Sheet.h>
#include "src/App/Property.h"
class SpreadsheetRenameProperty: public RenameProperty
{
protected:
void SetUp() override
{
RenameProperty::SetUp();
sheet = freecad_cast<Spreadsheet::Sheet*>(_doc->addObject("Spreadsheet::Sheet", "Sheet"));
}
void TearDown() override
{
_doc->removeObject(sheet->getNameInDocument());
RenameProperty::TearDown();
}
Spreadsheet::Sheet* sheet {};
};
std::string RenameProperty::_docName;
App::Document* RenameProperty::_doc {nullptr};
// Tests whether we can rename a property that is used in a spreadsheet
TEST_F(SpreadsheetRenameProperty, renameProperty)
{
// Arrange
sheet->setCell("A1", "=VarSet.Variable");
_doc->recompute();
auto* propSpreadsheet = static_cast<App::PropertyInteger*>(sheet->getPropertyByName("A1"));
// Assert before the rename
EXPECT_EQ(prop->getValue(), Value);
EXPECT_EQ(propSpreadsheet->getValue(), Value);
// Act
bool isRenamed = varSet->renameDynamicProperty(prop, "NewName");
_doc->recompute();
// Assert after the rename
EXPECT_TRUE(isRenamed);
EXPECT_STREQ(varSet->getPropertyName(prop), "NewName");
EXPECT_EQ(prop->getValue(), Value);
EXPECT_EQ(varSet->getDynamicPropertyByName("Variable"), nullptr);
EXPECT_EQ(varSet->getDynamicPropertyByName("NewName"), prop);
std::string cellContent;
sheet->getCell(App::CellAddress("A1"))->getStringContent(cellContent);
EXPECT_EQ(cellContent, "=VarSet.NewName");
EXPECT_EQ(propSpreadsheet->getValue(), Value);
}