You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@weex.apache.org by GitBox <gi...@apache.org> on 2020/06/09 15:52:16 UTC

[GitHub] [incubator-weex] qwertyI opened a new issue #3224: [Android][Coverage]Can i use istanbul/nyc collect weex android project code coverage ?

qwertyI opened a new issue #3224:
URL: https://github.com/apache/incubator-weex/issues/3224


   <!--
   Thanks for using Weex. Github Issue is only used for bur report and please follow the template to report the bug. 
   A good bug report should include the following information:
   -->
   
   <!-- Requirements: please go through this checklist before opening a new issue
     - * Review the documentation: https://weex.apache.org/index.html
     - * Search for existing issues: https://github.com/apache/incubator-weex/issues
     - * Use the latest Weex release: https://github.com/apache/incubator-weex/releases -->
   
   
   ## Describe the bug
   **this may not be a bug**
   i am new to Weex.
   Recently I used Istanbul to collect code coverage of weex android project.
   like thie 
   `
   fetch('http://10.96.2.20:8889/coverage/client', {
         method: 'POST',
         headers: {
           'Content-Type': 'application/json',
         },
         body: JSON.stringify(window.__coverage__)
       })
       .then(function () {
         native_logMessage('[Coverage] upload success')
       })
   `
   on Web side, it is worked.
   but window is not used on android.
   I would like to ask Weex if there are other ways for me to obtain coverage data
   <!-- A clear and concise description of what the bug is. -->
   
   ## To Reproduce
   <!-- Steps to reproduce the behavior.  Include a MCVE using http://dotwe.org/vue , crash stacktrace or  share your app project. Please follow the guidelines for providing a MCVE: https://stackoverflow.com/help/mcve -->
   
   ## Environment
   2.0.0-beta.31 (Core 2.0.0-beta.31)
   - @weex-cli/generator : v2.0.0-beta.16
   - @weex-cli/run : v2.0.0-beta.21
   - @weex-cli/device : v2.0.0-beta.2
   <!-- 
   * Device: [e.g. iPhone6]
   * OS: [e.g. iOS8.1]
   * Version [e.g. 22]
   * Build from source [e.g. yes/no] -->
   
   ## Expected behavior
   <!-- A clear and concise description of what you expected to happen. -->
   
   ## Screenshots
   <!-- If applicable, add screenshots to help explain your problem. -->
   
   ## Additional context
   <!-- Add any other context about the problem here.-->
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-weex] qwertyI edited a comment on issue #3224: [Android][Coverage]Can i use istanbul/nyc collect weex android project code coverage ?

Posted by GitBox <gi...@apache.org>.
qwertyI edited a comment on issue #3224:
URL: https://github.com/apache/incubator-weex/issues/3224#issuecomment-641113928


   > storage
   
   you mean, `weex.__coverage__` and `storage.__coverage__` ?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-weex] cnryb commented on issue #3224: [Android][Coverage]Can i use istanbul/nyc collect weex android project code coverage ?

Posted by GitBox <gi...@apache.org>.
cnryb commented on issue #3224:
URL: https://github.com/apache/incubator-weex/issues/3224#issuecomment-642552057


   `weex` is a global variable.  
   storage is a weex module.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-weex] cnryb commented on issue #3224: [Android][Coverage]Can i use istanbul/nyc collect weex android project code coverage ?

Posted by GitBox <gi...@apache.org>.
cnryb commented on issue #3224:
URL: https://github.com/apache/incubator-weex/issues/3224#issuecomment-641102420


   You can use `weex` instead of `window` . Or use [storage](https://weex.apache.org/docs/modules/storage.html).


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-weex] qwertyI commented on issue #3224: [Android][Coverage]Can i use istanbul/nyc collect weex android project code coverage ?

Posted by GitBox <gi...@apache.org>.
qwertyI commented on issue #3224:
URL: https://github.com/apache/incubator-weex/issues/3224#issuecomment-641113928


   > storage
   
   you mean, weex.__coverage__ and storage.__coverage__ ?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org