Add 10Cap valuation (closes #88) (#90)

* Add 10Cap valuation (closes #88).

* Delete the test that is not relevant anymore.

I expect the breaking of this test to be fixed by PR #92.

* Fix div id.

* Make the test test the *API*, not NVDA:)

Actually, the purpose of this commit is to bump the CI to run again,
because it failed with an error that seems intermittent, and very rare.

* Fix precision of 10Cap price.

Before: ten_cap("NVDA") = 10.834

After: ten_cap("NVDA") = 10.83

Author: kocielnik

isthisstockgood/DataFetcher.py

@@ -31,6 +31,7 @@ def fetchDataForTickerSymbol(ticker):
         'debt_equity_ratio',
         'margin_of_safety_price',
         'current_price'
+        'ten_cap_price'
   """
   if not ticker:
     return None
@@ -56,7 +57,9 @@ def fetchDataForTickerSymbol(ticker):
   margin_of_safety_price, sticker_price = \
       _calculateMarginOfSafetyPrice(msn_money.equity_growth_rates[-1], msn_money.pe_low, msn_money.pe_high, msn_money.eps[-1], five_year_growth_rate)
   payback_time = _calculatePaybackTime(msn_money.equity_growth_rates[-1], msn_money.last_year_net_income, msn_money.market_cap, five_year_growth_rate)
-  computed_free_cash_flow = round(float(msn_money.free_cash_flow[-1]) * msn_money.shares_outstanding)
+  free_cash_flow_per_share = float(msn_money.free_cash_flow[-1])
+  computed_free_cash_flow = round(free_cash_flow_per_share * msn_money.shares_outstanding)
+  ten_cap_price = 10 * free_cash_flow_per_share
   template_values = {
     'ticker' : ticker,
     'name' : msn_money.name if msn_money and msn_money.name else 'null',
@@ -68,6 +71,7 @@ def fetchDataForTickerSymbol(ticker):
     'cash': msn_money.free_cash_flow_growth_rates if msn_money and msn_money.free_cash_flow_growth_rates else [],
     'total_debt' : msn_money.total_debt,
     'free_cash_flow' : computed_free_cash_flow,
+    'ten_cap_price' : round(ten_cap_price, 2),
     'debt_payoff_time' : round(float(msn_money.total_debt) / computed_free_cash_flow),
     'debt_equity_ratio' : msn_money.debt_equity_ratio if msn_money and msn_money.debt_equity_ratio >= 0 else -1,
     'margin_of_safety_price' : margin_of_safety_price if margin_of_safety_price else 'null',

isthisstockgood/templates/home.html

@@ -7,11 +7,12 @@
     {% include "management.html" %}
     {% include "margin_of_safety.html" %}
     {% include "payback_time.html" %}
+    {% include "ten_cap.html" %}
     {% include "market_cap.html" %}
     <script>
       // Initialize all popover elements for the Bootstrap plugin.
       $(document).ready(function(){
         $('[data-toggle="popover"]').popover({ trigger: "hover" });
       });
     </script>
-	{% endblock %}
+	{% endblock %}

isthisstockgood/templates/js/search.js

@@ -122,6 +122,22 @@ $(document).ready(function() {
         colorCellWithBackgroundColor('#' + key, Color.red());
       }
 
+      // Update 10 Cap section
+      let ten_cap_key = 'ten_cap_price';
+      let ten_cap_field_id = '#' + ten_cap_key;
+      let current_price = data['current_price'];
+      updateHtmlWithValueForKey(data, ten_cap_key, /*commas=*/true);
+      if (!data[ten_cap_key]) {
+        colorCellWithBackgroundColor(ten_cap_field_id, Color.red());
+      }
+      if (current_price > data[ten_cap_key]) {
+        colorCellWithBackgroundColor(ten_cap_field_id, Color.red());
+      }
+      else {
+        colorCellWithBackgroundColor(ten_cap_field_id, Color.green());
+      }
+
+
       // Update Market Cap numbers
       updateHtmlWithValueForKey(data, 'average_volume', /*commas=*/true);
       let averageVolume = data['average_volume'];
@@ -212,4 +228,4 @@ function colorCellWithIDForZeroBasedRange(id, range) {
     }
 
     colorCellWithBackgroundColor(id, backgroundColor);
-}
+}

isthisstockgood/templates/json/stock_data.json

@@ -15,5 +15,6 @@
   "current_price" : {{ current_price }},
   "sticker_price" : {{ sticker_price }},
   "payback_time" : {{ payback_time }},
+  "ten_cap_price" : {{ ten_cap_price }},
   "average_volume" : {{ average_volume }}
-}
+}

isthisstockgood/templates/loading.html

@@ -90,7 +90,7 @@
 
             this.shadowRoot.append(style, wrapper);
 
-            this.animationTimeout = None;
+            this.animationTimeout = null;
         }
 
         static get observedAttributes() { return ['data-message']; }

isthisstockgood/templates/searchbox.html

@@ -4,11 +4,11 @@
       <div class="col-md-3 nopadding">
         <input
             type="text"
+            autofocus
             class="form-control"
             id="ticker"
             placeholder="Ticker Symbol"
-            maxlength=8
-            onkeydown="return validateInput(event)"/>
+            maxlength=8/>
       </div>
       <div class="col-md-1 nopadding">
         <input
@@ -32,4 +32,4 @@
 
 <script>
   {% include "js/search.js" %}
-</script>
+</script>

isthisstockgood/templates/ten_cap.html

@@ -0,0 +1,25 @@
+<div class="container pb-2 pt-2" id="Ten Cap">
+  <div class="row justify-content-center">
+    <button
+        type="button"
+        class="btn btn-raised btn-lg btn-primary"
+        data-toggle="popover"
+        title="10 Cap"
+        data-content="{% include 'txt/ten_cap_tooltip.txt' %}"
+    >10 Cap</button>
+  </div>
+
+    <div class="row pt-3 justify-content-center">
+    <div class="col-sm-5">
+      <table class="table table-bordered text-center">
+        <tr id="debt_equity_title_label">
+          <thead class="thead-light">
+            <th>10 Cap Price</th>
+          </thead>
+          <td id="ten_cap_price">
+            -
+          </td>
+        </tr>
+      </table>
+    </div>
+</div>

isthisstockgood/templates/txt/ten_cap_tooltip.txt

@@ -0,0 +1,2 @@
+Price at which the company is expected to return to you about 10% of your invested amount via Free Cash Flow (Owner's Earnings).
+Discussed in the book 'Invested' by Danielle Town.

tests/test_DataSources.py

@@ -32,15 +32,6 @@ def test_msn_money():
     assert data.last_year_net_income > 0.0
     assert data.total_debt >= 0.0
 
-def test_yahoo():
-    test_ticker = 'MSFT'
-    test_name = 'Microsoft Corp'
-
-    data = get_yahoo_data(test_ticker)
-
-    assert data.ticker_symbol == test_ticker
-    assert float(data.five_year_growth_rate) > 0.0
-
 def get_msn_money_data(ticker):
     data_fetcher = DataFetcher()
     data_fetcher.ticker_symbol = ticker

tests/test_api.py

@@ -18,7 +18,27 @@ def test_get_data():
 
     with app.test_client() as test_client:
         test_client = app.test_client()
+
         res = test_client.get('/api/ticker/nvda')
-        data = res.text
-        assert json.loads(data)['debt_payoff_time'] == 0
         assert res.status_code == 200
+
+        assert res.json['debt_payoff_time'] >= 0
+
+def test_get_ten_cap_price():
+    app = create_app(fetchDataForTickerSymbol)
+
+    with app.test_client() as test_client:
+        test_client = app.test_client()
+        res = test_client.get('/api/ticker/nvda')
+        assert res.json['ten_cap_price'] > 0
+
+def test_ten_cap_price_has_two_places_precision():
+    app = create_app(fetchDataForTickerSymbol)
+
+    with app.test_client() as test_client:
+        test_client = app.test_client()
+        res = test_client.get('/api/ticker/nvda')
+
+        price = res.json['ten_cap_price']
+
+        assert round(price, 2) == price