Skip to content

Conversation

@kio-watanabe
Copy link
Contributor

メソッドcob_delete_fileをCobolFile.java内に作成しました。

Copy link
Contributor

@yutaro-sakamoto yutaro-sakamoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cobj/typeck.cの多くの箇所が変更されていますが、空白やインデントの修正だけでしょうか。
https://github.com/opensourcecobol/opensourcecobol4j/pull/68/files#diff-ee8d6754c36011d2eae7a10a89115198ea7f4b977a1fa1538262ea97bc9d5479
もしプログラムの動作に影響を与える変更があるなら、どの箇所を変更したかを示しもらえますでしょうか。

@kio-watanabe
Copy link
Contributor Author

typeck.cの4193行目を以下のように変更しています。

cb_emit (cb_build_funcall_2 ("cob_delete_file", file, CB_FILE (file)->file_status));

cb_emit(cb_build_method_call_2("cob_delete_file", file, CB_FILE(file)->file_status));

@yutaro-sakamoto
Copy link
Contributor

今回の修正により、結果が失敗から成功に変わったテストケースはありますか?
もしあるなら、そのテストケースがCIで実行されるように修正してください。
もしないのであれば、今回の変更に対する新たなテストケースを追加してください。

@kio-watanabe
Copy link
Contributor Author

私のgithubアカウントの方では、jp-compat の Allow file delete (SEQUENTIAL) と Allow file delete (LINE SEQ)がokになっていました。
こちらの方ではテストがスキップされているのですが、テストが実行されるように修正する方法がわからないです。

@yutaro-sakamoto
Copy link
Contributor

yutaro-sakamoto commented Jan 19, 2023

https://github.com/kio-watanabe/opensourcecobol4j/actions/runs/3946732106/jobs/6754765182#step:12:63
これでしょうか。確かにこのCIではテストが成功していますね。

解決策としては、このブランチ(opensourcecobol:develop)をkio-watanabe:fix/file-controlにマージしてください。
その後、下記のコードを削除して、テストケースのスキップを解除してください。

AT_CHECK([${SKIP_TEST}])

AT_CHECK([${SKIP_TEST}])

このdevelopブランチのCIは過去のPull Requestにより改善しています。
詳細はリンク先を見てほしいのですが、失敗していたテストをすべてスキップし、スキップしていないテストが1つでも失敗すればCIも失敗するように変更しました。

おそらく、このブランチ(opensourcecobol:develop)の変更を取り込んでいないために、CIの結果に差異が現れたのだと思います。

@kio-watanabe
Copy link
Contributor Author

Allow file delete (SEQUENTIAL) と Allow file delete (LINE SEQ) が実行されるように修正しました。

@yutaro-sakamoto yutaro-sakamoto merged commit 95d121c into opensourcecobol:develop Jan 19, 2023
@kio-watanabe kio-watanabe deleted the fix/file-control branch September 6, 2023 06:40
kio-watanabe pushed a commit to kio-watanabe/opensourcecobol4j that referenced this pull request Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants