search

slrtbtfs / graphfind-rs

Library for Finding Patterns in graphs
Apache License 2.0
5 stars 0 forks source link

Reorganize tests - [merged] #59

Closed slrtbtfs closed 1 year ago

slrtbtfs commented 1 year ago

_Merges warningfixes -> main

Fixes the false positive warnings (see https://gitlab.informatik.uni-ulm.de/se/graphquery/rustgql/-/commit/2144cced7311225cdf44195217ce26f3329e53d7) emitted by cargo, places the functions used by multiple tests more visible in a common module, and changes the Person graph type, which fixes #38.

slrtbtfs commented 1 year ago

We had that one twice.

slrtbtfs commented 1 year ago

requested review from @bjthehun

slrtbtfs commented 1 year ago

In GitLab by @bjthehun on Dec 3, 2022, 14:37

Commented on tests/petgraph.rs line 5

Ok.

slrtbtfs commented 1 year ago

In GitLab by @bjthehun on Dec 3, 2022, 14:37

resolved all threads

slrtbtfs commented 1 year ago

In GitLab by @bjthehun on Dec 3, 2022, 14:37

Looks good.

slrtbtfs commented 1 year ago

In GitLab by @bjthehun on Dec 3, 2022, 14:41

approved this merge request

slrtbtfs commented 1 year ago

In GitLab by @bjthehun on Dec 3, 2022, 14:41

mentioned in commit 5c1cba2b332544f36aa17cb961bb27c06e3413f8