fact-bounty icon indicating copy to clipboard operation
fact-bounty copied to clipboard

Add Travis CI (React)

Open JosephSemrai opened this issue 5 years ago • 3 comments

Added Travis configuration for React application.

Requires the setting of repository variables for the React tests and deployment to GitHub pages.

JosephSemrai avatar Dec 26 '19 21:12 JosephSemrai

Hi, were the repository environment variables set? It should build given valid values.

  ✕ renders without crashing (199ms)
  ● renders without crashing
    TypeError: Cannot read property 'initialize' of undefined
      15 |     _me
      16 | 
    > 17 |   OAuth.initialize(initialize_key)
         |         ^
      18 |   OAuth.setOAuthdURL(OAUTH_URL)
      19 | 
      20 |   function getCachedToken() {

Looking at the output of the failed Travis build. The issue stems from the testing suite and possibly the lack of repository variables as the Travis configuration launches the tests appropriately.

JosephSemrai avatar Dec 27 '19 03:12 JosephSemrai

There is a small issue with regard to OAuth library but there were no build issues in the past as to my knowledge. Anyway we can't merge this until the build test passes. @JosephSemrai is it possible to make changes to avoid this issue ?

root5533 avatar Dec 28 '19 19:12 root5533

I believe there were no build errors in the past as the React application was not being built with it requiring the environment variables. If setting the environment variables in the repository settings (with someone who has permission) is not possible, I will try to look for another solution.

Thanks for the feedback.

JosephSemrai avatar Dec 30 '19 04:12 JosephSemrai