Skip to content
Snippets Groups Projects
Commit ab931fa8 authored by florian's avatar florian
Browse files

MAINT: Remove some todos

parent 13cb4e76
No related branches found
No related tags found
1 merge request!13ENH: Add datatypes and value classes to Extern C interface
Pipeline #12282 passed
Pipeline: caosdb-cppinttest

#12285

    This commit is part of merge request !13. Comments created here will be created in the context of that merge request.
    ......@@ -1045,9 +1045,6 @@ CAOSDB_PROPERTY_SET(id, id, wrapped_property->SetId(std::string(id));)
    // TODO(fspreck)
    // CAOSDB_PROPERTY_SET(datatype, datatype,
    // wrapped_property->SetDataType(std::string(datatype));)
    // TODO(fspreck)
    // CAOSDB_PROPERTY_SET(importance, importance,
    // wrapped_property->SetImportance(std::string(importance));)
    ERROR_RETURN_CODE(GENERIC_ERROR,
    int caosdb_entity_property_set_importance(caosdb_entity_property *property,
    const char *importance),
    ......
    ......@@ -221,7 +221,6 @@ TEST_F(test_ccaosdb, test_property) {
    caosdb_entity_property_set_name(&property, "some_name");
    // TODO(fspreck)
    // caosdb_entity_property_set_datatype(&property, "TEXT");
    // TODO(fspreck)
    caosdb_entity_property_set_importance(&property, "FIX");
    caosdb_entity_property_set_unit(&property, "some_unit");
    caosdb_entity_property_set_string_value(&property, "some_value");
    ......@@ -237,7 +236,6 @@ TEST_F(test_ccaosdb, test_property) {
    // caosdb_entity_property_get_datatype(&property, out);
    // EXPECT_EQ(strcmp(out, "TEXT"), 0);
    // TODO(fspreck)
    caosdb_entity_property_get_importance(&property, &out);
    EXPECT_EQ(strcmp(out, "FIX"), 0);
    ......
    0% Loading or .
    You are about to add 0 people to the discussion. Proceed with caution.
    Finish editing this message first!
    Please register or to comment