Testing changes to `bin_prot`/`ppx_bin_prot`?

Ecosystem
,
1

I have developed an experimental change to bin_protto help with migrating data across schema versions. This change affects the API of bin_prot, insofar as 'a reader becomes (’a, 'ctx) reader and bin_read_t takes an additional argument.

I’m now trying to patch ppx_bin_prot to be able to automatically @@derive readers. I have code that builds, and probably doesn’t work yet, but I’m at a loss on how to test this.

Any attempt I’ve made to use my modified ppx_bin_prot requires gel, which uses ppx_bin_prot, but somehow ends up using my new bin_prot with the existing ppx_bin_prot.

Has anybody attempted to contribute to bin_prot/ppx_bin_prot? If so, how did you handle testing?

2

Cc @Jonathan whom I believe has successfully contributed to ppx_bin_prot in the past.

3

I did contribute a small fix to `ppx_bin_prot` a few years ago. The fix was straightforward enough that I did not have to do testing beyond running the existing test suite and providing a handful of new examples. I do not remember running into any issue running the test suite but this was a long time ago. I am not sure I understand what tests you are trying to run here and how exactly you are running into problems.