プリロードのテスト等修正 #141

kamataryo · 2022-01-19T08:45:03Z

ローカルファイル参照が動作していなかった（すみません..）のでこのバグを修正
README にプリロードオプションについて記載
外部に通信していないことをテストするようにテストを修正

keichan34 · 2022-01-20T02:41:43Z

src/main-node.ts

@@ -28,7 +29,9 @@ export const preload = async () => {
  // file:// でローカルにダウンロードした zip ファイルを参照する。
  // https://github.com/geolonia/japanese-addresses のリポジトリと同じ構造を持つものを想定
  if (currentConfig.japaneseAddressesApi.startsWith('file://')) {
-    zipBuffer = fs.readFileSync(currentConfig.japaneseAddressesApi)
+    zipBuffer = fs.readFileSync(
+      currentConfig.japaneseAddressesApi.replace('file:/', ''),


replace(/^file:\// の方がいいと思います

@keichan34 ありがとうございます修正しました 🙇

kamataryo added 3 commits January 19, 2022 16:38

Fix local data source bug

fdd5432

Fix test

aac6fd7

Update sample code in readme

9749018

kamataryo requested a review from keichan34 January 19, 2022 08:45

keichan34 reviewed Jan 20, 2022

View reviewed changes

Fix regex to convert file URL to path

5df0171

kamataryo requested a review from keichan34 January 20, 2022 02:48

keichan34 approved these changes Jan 20, 2022

View reviewed changes

kamataryo merged commit 0bd6100 into master Jan 20, 2022

kamataryo deleted the enhance-testing branch January 20, 2022 02:49

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

プリロードのテスト等修正 #141

プリロードのテスト等修正 #141

kamataryo commented Jan 19, 2022

keichan34 Jan 20, 2022

kamataryo Jan 20, 2022

プリロードのテスト等修正 #141

プリロードのテスト等修正 #141

Conversation

kamataryo commented Jan 19, 2022

keichan34 Jan 20, 2022

Choose a reason for hiding this comment

kamataryo Jan 20, 2022

Choose a reason for hiding this comment