Closed RobStallion closed 5 years ago
@Danwhy I do not think that this PR is complete (at all) but I think that it might be in a place that it can be merged into the adapter branch.
Please let me know if you have any thoughts on this.
I think if we add some tests, we may get a better idea of how we want it to actually work/what makes sense with regards to duplicate data etc.
You'll need to delete or comment out all the old tests that have use Alog
in them to run any tests you write. Take a look at the update test on my update
branch if you need any help getting started.
Testing the insert functions has 'brought to light' some things that we could (and maybe should) change in the future. Please see this comment for more info on this.
45
"alog-ifies" insert functionality, adding a CID made from the params and a unique entry_id from a substring of the CID.
Adds tests for insert function
This step, as it is currently, allows for duplicate data to be inserted. Comments have been added to code for my thoughts on this matter.