Skip to content

Commit a4872ab

Browse files
committed
feat:
- fixing test - adding a update
1 parent 5e43f93 commit a4872ab

File tree

2 files changed

+33
-8
lines changed

2 files changed

+33
-8
lines changed

sqlRelational/db_test.go

+23-6
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ func TestDB(t *testing.T) {
4747

4848
websiteReturned, err := sqlDB.FetchWebsite(website.Url)
4949
require.NoError(t, err)
50-
require.Equal(t, website, websiteReturned)
50+
require.Equal(t, &website, websiteReturned)
5151
})
5252

5353
t.Run("AddPage", func(t *testing.T) {
@@ -63,21 +63,38 @@ func TestDB(t *testing.T) {
6363

6464
pageReturned, err := sqlDB.FetchPage(page.Url)
6565
require.NoError(t, err)
66-
require.Equal(t, page, pageReturned)
67-
66+
require.Equal(t, &page, pageReturned)
6867
})
6968

7069
t.Run("Fetch page empty", func(t *testing.T) {
7170
pageReturn, err := sqlDB.FetchPage("does not exist")
7271
require.NoError(t, err)
73-
require.Equal(t, reflect.DeepEqual(pageReturn, site.Page{}), true)
74-
72+
require.Equal(t, reflect.DeepEqual(pageReturn, &site.Page{}), true)
7573
})
7674

7775
t.Run("Fetch website empty", func(t *testing.T) {
7876
pageReturn, err := sqlDB.FetchWebsite("does not exist website")
7977
require.NoError(t, err)
80-
require.Equal(t, reflect.DeepEqual(pageReturn, site.Website{}), true)
78+
require.Equal(t, reflect.DeepEqual(pageReturn, &site.Website{}), true)
79+
})
80+
81+
t.Run("update website", func(t *testing.T) {
82+
page := site.Page{
83+
Url: "http://www.google.com",
84+
Title: "Google",
85+
Body: "Search Engine",
86+
BaseURL: "http://www.google.com",
87+
}
88+
89+
website := site.Website{
90+
Url: "http://www.google.com",
91+
ProminenceValue: 1,
92+
}
93+
err := sqlDB.UpdateWebsite(page, website)
94+
require.NoError(t, err)
95+
returnDB, err := sqlDB.FetchWebsite(website.Url)
96+
require.NoError(t, err)
97+
require.Equal(t, returnDB.ProminenceValue, website.ProminenceValue+1)
8198

8299
})
83100
}

sqlRelational/update.go

+10-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package sqlRelational
22

33
import (
4+
"fmt"
45
"reflect"
56
"webcrawler/site"
67
)
@@ -13,10 +14,17 @@ func (c *SqlDB) UpdateWebsite(page site.Page, website site.Website) error {
1314
}
1415
if reflect.DeepEqual(websiteDB, &site.Website{}) && err == nil {
1516
println("websiteNot found")
16-
websiteDB = &website
17+
err = c.AddWebsite(website)
18+
if err != nil {
19+
return err
20+
}
21+
return nil
1722
}
23+
websiteDB.ProminenceValue += 1
24+
queryString := fmt.Sprintf("UPDATE website SET promancevalue = promancevalue + 1 WHERE baseurl = '%s' ", website.Url)
25+
_, err = c.Client.Query(queryString)
1826

19-
return nil
27+
return err
2028
}
2129

2230
func (c *SqlDB) UpdatePage(page site.Page) error {

0 commit comments

Comments
 (0)