Skip to content

Commit

Permalink
GH-37181: [MATLAB] Remove outdated test class tArrowCppCall.m (#37185)
Browse files Browse the repository at this point in the history
### Rationale for this change

Now that `featherread` and `featherwrite` have been re-implemented in terms of the new APIs (`matlab.internal.io.feather.Reader` in #37046 and `matlab.io.internal.feather.Writer` in #37045), we can start removing the MEX code. To avoid submitting a large pull request, we should break the work down into several smaller, incremental steps.

The first step should involve deleting the outdated test class `tArrowCppCall.m`. This test class will no longer be relevant once the MEX source code is removed.

### What changes are included in this PR?

1. Deleted the outdated test class `tArrowCppCall.m`

### Are these changes tested?

No tests necessary.

### Are there any user-facing changes?

No.

### Future Directions

1. Move the test cases in `test/tfeathermex.m` to `test/arrow/internal/io/feather/tRoundtrip.m`. These test cases directly call the MEX functions, so we will re-write the tests to utilize the `arrow.internal.io.feather.Reader` and `arrow.internal.io.feather.Writer` classes directly.
2. Delete `test/arrow/util/createVariablesAndMetadataStruct.m`
3. Delete `test/arrow/util/createTable.m`
4. Delete `test/arrow/util/featherMEXRoundTrip.m`
5. Delete `test/arrow/util/featherRoundTrip.m`
6. Remove MEX source code.
* Closes: #37181

Authored-by: Sarah Gilmore <sgilmore@mathworks.com>
Signed-off-by: Kevin Gurney <kgurney@mathworks.com>
  • Loading branch information
sgilmore10 committed Aug 15, 2023
1 parent 5c23938 commit f3dec5c
Showing 1 changed file with 0 additions and 61 deletions.
61 changes: 0 additions & 61 deletions matlab/test/tArrowCppCall.m

This file was deleted.

0 comments on commit f3dec5c

Please sign in to comment.