New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hifive本体やテストコードで、グローバル領域に漏れている変数をチェックして対応する #289

Closed
fukudayasuo opened this Issue Mar 20, 2014 · 0 comments

Comments

Projects
None yet
2 participants
@fukudayasuo

fukudayasuo commented Mar 20, 2014

CheckForGlobalsを有効にしてテストランナーを回すと、以下のテストでグローバル領域に値が漏れている様子。
確認して対応する。

  • obj#ns.名前空間作成 (h5.u.obj.ns) VM1679:2
  • obj#ns.名前空間作成-ドット区切りでネスト VM1679:2
  • obj#expose.h5test1.exposeにオブジェクトを公開する VM1679:2
  • Async - commonFailHandlerの動作.alwaysでfailコールバックを登録してreject()された場合、commonFailHandlerが呼ばれないこと VM1679:2
  • [build#min;browser#ie:6]JQMManager - define.init()を実行せずにdefine()を実行して、遷移を行う VM1679:2
  • Controller - expose.h5.core.expose()を実行 VM1679:2
  • Controller.イベントハンドラの{}記法でオブジェクトを指定する時に2階層以上下のオブジェクトを指定できるか VM1679:2
  • Controller.テンプレートのロードが失敗したとき、commonFailHandlerのthisはコントローラインスタンス、引数はview.loadのエラーオブジェクトであること VM1679:2
  • createManager.データモデルマネージャの作成 名前空間指定が空文字の場合はwindow直下ににマネージャを作成できること VM1679:2
  • createModel - ディスクリプタのチェック.baseにデータモデルを指定した時、継承元のスキーマは変わらないこと VM1679:2
  • ObservableArray Arrayメソッド 非破壊系.concat VM1679:2
  • [browser#ie:all|ie-wp:all|ff:all]H5Api - Web SQL Database.スタブを使ったテスト。各エラーを取得する

@simdy simdy added this to the v1.1.10 milestone Mar 20, 2014

@simdy simdy added bug and removed bug labels Mar 20, 2014

fukudayasuo pushed a commit to hifive-labs/hifivemain that referenced this issue Mar 25, 2014

fukudayasuo
#289 グローバル漏れの対応を行いました。
・varが無くてグローバルに漏れている箇所を修正しました。
・deleteProperty()関数を作成し、グローバルに変数を公開するテストについて、終了時にdeleteを行うようにしました。
・テストで使用するh5version0.0.1を、グローバル漏れ対応後のものに差し替えました。

@simdy simdy added the bug label Mar 25, 2014

@simdy simdy closed this Mar 25, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment