When is method schema_status(const storage_impl<index_t<Cols...>, Tss...>&, sqlite3*, bool) called?

[juandent]

Hi

There is a method in class storage_t<...> (see storage.h) that is not invoked by any of the unit-tests. Its signature is:

template<class... Tss, class... Cols>
            sync_schema_result schema_status(const storage_impl<index_t<Cols...>, Tss...>&, sqlite3*, bool) {
                return sync_schema_result::already_in_sync;
            }

I need to find when this is called to make sure the corrections to schema_status() are complete and correct. (see PR #1010)

Help will be greatly appreciated....

Juan

[fnc12]

it is called when you call sync_schema a storage with an index(es)

[juandent]

any particular unit-test that exercises this method?
I need to test it to make sure everything is ok

[fnc12]

dunno. The amount of unit-tests is so huge that I cannot remember all of them. Try to search using make_index inside unit-tests code or create your own. You can copy and paste code from index.cpp example

[juandent]

the function schema_status(const storage_impl<index_t<Cols...>, Tss...>&, sqlite3*, bool)

is not called by this code:

    struct User {
        int id = 0;
        std::string name;
    };
    auto storage = make_storage("", make_index("user_name", &User::name),
        make_table("users",
            make_column("id", &User::id, primary_key(), autoincrement()),
            make_column("name", &User::name)));
    storage.sync_schema(true);

how can I test that function?

[fnc12]

try to comment it out and compile. Does it compile? Also try to call sync_schema_simulate

[juandent]

Will try! Thanks

…

Sent from my iPhone

On 18 Apr 2022, at 6:35 AM, Yevgeniy Zakharov ***@***.***> wrote:  try to comment it out and compile. Does it compile? Also try to call sync_schema_simulate — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.

[juandent]

```C++`
storage.sync_schema_simulate(true);

succeeds in calling the function!

Thanks Eugene!