bolero-fk / topologymatching Goto Github PK

GameConfigクラスのコンストラクタなどの引値に型指定ができていないので修正する
関数Validateが小文字から始まっていないので修正する
関数Validateにprivate修飾子がついていないので修正する

各クラスのコンストラクタに入力された値に対するvalidationを追加する

異常系のテストコードを追加する。

主にmain.tsやgameEngine.tsについてリファクタリングを行う

現在mainクラスでhtml要素の初期化が実行されているが、初期化専用のクラスを用意しクラスを通して初期化するように変更する。

htmlのページにタイトルタグが設定されていないので設定する

少しロジックがややこしく、行数が多くなってきたのでTopologyCalculatorのリファクタリングを行う

documentがglobal変数を使用しているため、少しテストが難しくなっているのでInitializerクラスのコンストラクタでdocumentを受け取るとりメンバ変数としてdocumentを操作するように変更する

github pagesのワークフローが正しく実行されていないので修正を行う

実行されていないactionは以下のようになっている
https://github.com/Bolero-fk/TopologyMatching/actions/runs/6521598560

カードを2枚選択した際に数秒待った後にカードが裏向きになるが、裏向きになるまえにrestart gameボタンを押すと挙動がおかしくなるので修正する

正方形でない画像を入力するとTopologyStatusCalculatorの処理が落ちるバグ

typescriptで書かれたコードを変更したプルリクが発生した際に自動でテストを実行するように変更する

WSL2を使っている関係でZone.Identifierが一部入ってしまっているのでそのファイルの削除と.gitignoreへの登録を行う

typescriptの部分で一部の関数がPascal Caseになっているので修正を行う

jestを実行した際にいくつかwarningが表示されるので修正を行う

C#のコードについてコード規則に一貫性を持たせるために.editorconfigを追加する。
editorconfigファイルは以下のリンク先のものをベースに使用する
https://github.com/RehanSaeed/EditorConfig/blob/main/.editorconfig

MainFormクラスからsvgを読み込む処理や保存する処理を分割する

github pagesのデプロイアクションの名称がDeploy static content to PagesとなっているのでDeploy github pagesに変更する

関数のコメントが不足しているものがいくつかあるのでコメントの見直しを行う

使われているのにpackage.jsonに記述が不足しているパッケージが存在するのでpackage.jsonを更新する

ファイルによって改行コードがLFの場合とCRLFの場合があるのでLFに統一する

リーダブルコードにしたがって全体的にリファクタリングを行う

現在稼働しているテスト用のワークフローとgithub pagesのリリース用のワークフローが全ての変更に対してトリガーが設定されているので、関係のあるファイルが変更されたタイミングでワークフローが発火するように変更する

テストを容易にするためにjQueryを使用する箇所のInterface化を行う

現状ではjsonの入力値に型制約が存在しないので制約を追加する。

Initializerクラスのテストコードを追加する

テストを容易にするためにGameControllerの引数の型として使われるcardDomをICardDomに変更する

GameConfigクラスに入力された値が不正な値でないかを確かめる処理を追加する

CardクラスのテストをしやすいようにDOM操作とゲームロジックを分離する

typescript で書かれた部分に対してテストコードを追加する
テストフレームワークにはJestを使用する。

Readmeの見栄えをよくするためにBadgeを追加する

リファクタリングとして
GridクラスとPosクラスについて別のファイルとして分割する

cssファイルのコメントが英語になっているので他の部分と合わせるために日本語で書き直す

テストを容易にするために変数の状態を取得する関数を追加する

現在typescriptコードに関係するファイルのScriptsにはtsのコードやそれをコンパイルしたjsのコード、configコードが乱雑に保存されていてわかりづらくなっているのでファイル構成の見直しを行う

github actionsを使用してコードカバレッジサマリーの修飾をおこなう。
github actionsは以下を使用する
https://github.com/marketplace/actions/code-coverage-report-action

テストコードを追加する
テストフレームワークにはxUnitを使用する

MainFormのユニットテストを追加する

main関数からゲーム操作に関する処理について分割を行う

テストを容易にするためにmainクラスに存在するwindow.onloadの実行をhtmlへ移動する

tscを実行すると以下のエラーが生じるので修正を行う

gameEngine.ts:76:27 - error TS2802: Type 'IterableIterator' can only be iterated through when using the '--downlevelIteration' flag or with a '--target' of 'es2015' or higher.

76 for (const key of cardGroups.keys()) {

現在、新しくプルリクされるとDotNetTestのワークフローを実行しプルリクエストにコメントするようにしているが、コメントをする処理が正しく実行されないバグ
#26 のプルリクで確認ができる。
エラー内容は以下のようになっている

Run marocchino/sticky-pull-request-comment@v2
with:
recreate: true
path: coverage_report/Summary.txt
append: false
delete: false
only_create: false
only_update: false
hide_details: false
hide: false
hide_and_recreate: false
hide_classify: OUTDATED
ignore_empty: false
skip_unchanged: false
follow_symbolic_links: false
GITHUB_TOKEN: ***
env:
DOTNET_ROOT: C:\Program Files\dotnet
Error: Either message or path input is required

全ての画素が黒色の画像を入力すると図形の穴状況は{0}が正しい出力になるが{}が出力されるバグ

workflowの名前が"***Test with Coverage Report"と冗長なので修正する。

リポジトリ内で使用されているTopologyCardRegistrarはTopologyCardRegisterの方が適しているのでファイル名の変更を行う。
ソースコードの変更と混在するとrenameの扱いがうまくいかずコミットの差分がみづらくなるのでこの変更だけ本issueで行う。

現在デプロイされているページを開くと以下のエラーが発生し落ちてしまっているので修正する
Uncaught SyntaxError: The requested module './cardDom.js' does not provide an export named 'CardDom' (at main.js:1:10)

GameConfigクラスが思ったよりも大きくなりそうなのでgameElementInitializer.tsから分離を行う