Update lab files

Author: gj

.gitignore

@@ -33,6 +33,6 @@ jspm_packages/
 # macOS system directory file
 .DS_Store
 
-# Learn.co files
-.results.json
-learn.auth.data.json
+# Package lockfiles
+package-lock.json
+yarn.lock

.learn

@@ -1,7 +1,4 @@
 tags:
-  - learn.co
-  - flatiron school
-  - javascript
   - variables
   - let
   - const

README.md

@@ -1,24 +1,18 @@
-JavaScript Variables Lab
----
+# Variables Lab
 
 ## Objectives
-
 - Practice using the `const` and `let` variables in JavaScript
 
 ## Instructions
-
 Ok, so this is your first lab.  You'll notice a few new things in this lesson that we haven't encountered before. Don't worry, we'll walk you through them.
 
 ### Tests...
-
 The first new thing you'll notice is tests.  When we want to run an experiment, we need to develop a hypothesis and we need to test it.  In programming, we run tests serve as our experiment to verify that programs behave the way we think they do. Tests help us identify bugs, and they give us a sense of the health of our applications.
 
 Here, we use tests as teaching tools. Just like in a normal coding environment, we use tests to describe the program's behavior. You are in charge of getting the tests to pass.
 
 ### Structure
-
 The structure of this lab — where its files and folders are located — looks roughly like the following:
-
 ```
 ├── CONTRIBUTING.md
 ├── LICENSE.md
@@ -33,7 +27,6 @@ The structure of this lab — where its files and folders are located — looks
 All labs will more or less have the same structure. (And READMEs, for that matter, will still have CONTRIBUTING.md, LICENSE.md, and README.md files.)
 
 ### Code-along
-
 For now, open up `index.js` in your text editor. If you're using the Learn IDE, click the "Open" button in the top right hand corner of the lesson.  If you open up that "js-basic-variables-lab" folder, you'll see a list of files (along with a test/ directory). Click `index.js`, and it will open in the editor.
 
 In `index.js`, you should see, well, nothing. We'll fix that soon.
@@ -43,27 +36,26 @@ Now open up `test/index-test.js`. Hey, there's something! What's all of this stu
 **Note: The `test/index-test.js` has great info that we want to look at, but do not edit this file otherwise you may have extra difficulty passing this lab.**
 
 A few lines down in the `test/index-test.js` file you will see:
-
 ```js
-describe('Using Variables', () => {
+describe('index.js', function () {
   // there's stuff in here, too
-})
+});
 ```
 
 `describe` is a function provided by our test library, Mocha.  The word is basically used to hold our tests.  You can see that after the word `describe` is a description about our tests.  Tests are used as a way to document the behavior of a function to developers, and you can see that examples of that after the words `describe`.  For example, the next word `describe` is followed by the word `companyName` name.  Here the test is telling us that the tests that come after words will be about `companyName`.  Then comes the word `it`, where you see the following:
 
 ```js
-it('is set as Scuber', () => {
+it('is set as Scuber', function () {
   // tests are here
-})
+});
 ```
 
 So this is telling us that the `companyName` should be set to `Scuber`.  Finally, filling in the missing part of the `it` code, we see:
 
 ```js
-it('is set as Scuber', () => {
-  expect(companyName).to.equal('Scuber')
-})
+it('is set as Scuber', function () {
+  expect(companyName).to.equal('Scuber');
+});
 ```
 
 Here, we can see that it expects `companyName` to equal `Scuber`.  That `expect` and `to.equal` are essentially doing the same thing as `companyName == 'Scuber'`.  In other words, `expect(companyName).to.equal('Scuber')` is running code that will have this first test pass if `companyName` equals `Scuber` and fail if it does not.
@@ -79,40 +71,31 @@ Running the `learn` command will open up a new tab on your browser, showing the
 To get our first test to pass, we can open up our `index.js` file, and write the following:
 
 ```js
-
-let companyName = 'Scuber'
-
+let companyName = 'Scuber';
 ```
 
-Great, our first test is now passing. Except the second test is also about `companyName` and it is not.  It's not passing because, it expects a change to `companyName` to throw a `TypeError`.  It sounds like it wants `companyName` to be declared using a different keyword than the `let` keyword - it needs a keyword that is used for variables that can't be changed...  
+Great, our first test is now passing. Except the second test is also about `companyName` and it is not.  It's not passing because, it expects a change to `companyName` to throw a `TypeError`.  It sounds like it wants `companyName` to be declared using a different keyword than the `let` keyword - it needs a keyword that is used for variables that can't be changed...
 
 Ok, so we'll let you work through the problems below.  But in summary here is your workflow for a lab:
 
 1. Run `learn`.
-
 2. Read the errors; vocalize what they're asking you to do.
-
 3. Write code; repeat steps 1 and 2 often until a test passes.
-
 4. Repeat as needed for further tests.
-
 5. Run `learn submit` when finished!
 
-
 ## Working Through The Problems
-
 If you open up `test/indexTest.js`, you will see the tasks in front of you.
 
-+ `companyName` - Inside the `test/indexTest`, look inside of the word `describe` where the tests are trying to indicate that this test is describing the `companyName` variable.  The `it` word that comes afterwords, tells us the features of `companyName`.  In the first `it` function call, it says that `it` (companyName) `is set as Scuber`.  In the next line, you can see that the test checks to make sure this occurs by seeing if `companyName` equals `Scuber`.  So this means that you need to go to your `index.js` file and declare a variable named `companyName` and set it equal to `Scuber`.  Once you do that, if `learn` is running, you will see the first test in this lab as passing.  
++ `companyName` - Inside the `test/indexTest`, look inside of the word `describe` where the tests are trying to indicate that this test is describing the `companyName` variable.  The `it` word that comes afterwords, tells us the features of `companyName`.  In the first `it` function call, it says that `it` (companyName) `is set as Scuber`.  In the next line, you can see that the test checks to make sure this occurs by seeing if `companyName` equals `Scuber`.  So this means that you need to go to your `index.js` file and declare a variable named `companyName` and set it equal to `Scuber`.  Once you do that, if `learn` is running, you will see the first test in this lab as passing.
 
 In the next `it` function call, we are still describing `companyName`.  This time, it says it `raises error if the companyName is changed`. The next line of code tests this.  It's ok if some of the code in that line are confusing.  Just know that the code attempts to change `companyName` to a different value, and that this reassignment to throw an error.  So you need to make sure that you are using the correct variable type such that reassigning the variable would throw an error.
 
 + `mostProfitableNeighborhood` - Here we need to declare another variable, `mostProfitableNeighborhood` and assign it to `Chelsea`.  In the next `it` function call, you can see that our tests ensure that `mostProfitableNeighborhood` does not throw an error when reassigned.  So you need to make sure that you are using the correct variable type such that reassigning the `mostProfitableNeighborhood` would not throw an error.
 
-+ `companyCeo` - Here, we are getting more practice with declaring variables.  Once again, a reassignment should not throw an error.  
++ `companyCeo` - Here, we are getting more practice with declaring variables.  Once again, a reassignment should not throw an error.
 
 ## Resources
-
 - [MDN: Let](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let)
 - [MDN: Const](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/const)
 

index.html

@@ -21,8 +21,6 @@
     <!-- Include Chai as the matcher library. -->
     <script src="node_modules/chai/chai.js"></script>
 
-
-
     <!-- Include the student's JavaScript file. -->
     <script src="index.js"></script>
 
@@ -31,8 +29,5 @@
 
     <!-- Include Flatiron School's test runner to push results to Learn. -->
     <script src="node_modules/learn-browser/learnBrowser.min.js"></script>
-
-    <!-- Run the test suite automatically when the page loads. -->
-    <script>LearnBrowser.test();</script>
   </body>
 </html>

index.js

@@ -1 +1,5 @@
+const companyName = 'Scuber';
 
+let mostProfitableNeighborhood = 'Chelsea';
+
+let companyCeo = 'Susan Smith';

package.json

@@ -1,8 +1,12 @@
 {
   "name": "js-variables-lab",
-  "version": "0.2.0",
+  "version": "0.1.0",
   "description": "JavaScript Variables Lab for Learn.co",
   "main": "index.js",
+  "scripts": {
+    "postinstall": "gem install learn-co",
+    "test": "node_modules/browser-sync/bin/browser-sync.js start --config node_modules/learn-browser/bs-config.js"
+  },
   "repository": {
     "type": "git",
     "url": "git+ssh://git@github.com/learn-co-curriculum/js-basics-variables-lab.git"
@@ -23,6 +27,6 @@
   },
   "homepage": "https://github.com/learn-co-curriculum/js-basics-variables-lab#readme",
   "devDependencies": {
-    "learn-browser": "^0.1.6"
+    "learn-browser": "^0.1.13"
   }
 }

test/indexTest.js

@@ -1,33 +1,33 @@
 const expect = chai.expect;
 
-describe('Using Variables', function() {
-  describe('companyName', function() {
-    it('is set as Scuber', () => {
-      expect(companyName).to.equal('Scuber')
-    })
+describe('index.js', function () {
+  describe('companyName', function () {
+    it('is set as Scuber', function () {
+      expect(companyName).to.equal('Scuber');
+    });
 
-    it('raises error if the companyName is changed', () => {
-      expect(() => { companyName = 'specialCompany'}).to.throw(TypeError)
-    })
-  })
+    it('raises error if the companyName is changed', function () {
+      expect(function () { companyName = 'specialCompany' }).to.throw(TypeError);
+    });
+  });
 
-  describe('mostProfitableNeighborhood', function() {
-    it('is declared as equal to Chelsea', function() {
-      expect(mostProfitableNeighborhood).to.equal('Chelsea')
-    })
+  describe('mostProfitableNeighborhood', function () {
+    it('is declared as equal to Chelsea', function () {
+      expect(mostProfitableNeighborhood).to.equal('Chelsea');
+    });
 
-    it('does not raise error if the mostProfitableNeighborhood is changed', function(){
-      expect(() => { mostProfitableNeighborhood = 'Upper West Side'}).to.not.throw(TypeError)
-    })
-  })
+    it('does not raise error if the mostProfitableNeighborhood is changed', function () {
+      expect(function () { mostProfitableNeighborhood = 'Upper West Side' }).to.not.throw(TypeError);
+    });
+  });
 
-  describe('companyCeo', function() {
-    it('is declared as equal to Susan Smith', function() {
-      expect(companyCeo).to.equal('Susan Smith')
-    })
+  describe('companyCeo', function () {
+    it('is declared as equal to Susan Smith', function () {
+      expect(companyCeo).to.equal('Susan Smith');
+    });
 
-    it('does not raise error if the companyCeo is changed', function(){
-      expect(() => { companyCeo = 'Lauren Hart'}).to.not.throw(TypeError)
-    })
-  })
-})
+    it('does not raise error if the companyCeo is changed', function () {
+      expect(function () { companyCeo = 'Lauren Hart' }).to.not.throw(TypeError);
+    });
+  });
+});