Merge branch 'master' into check-format

Author: st0012

README.md

@@ -3,7 +3,7 @@
 
 [![](https://goby-slack-invite.herokuapp.com/badge.svg)](https://goby-slack-invite.herokuapp.com)
 [![Discord](https://img.shields.io/discord/678892628955103232?label=discord)](https://discord.gg/SS5HbYN)
-[![Build Status](https://travis-ci.org/goby-lang/goby.svg?branch=master)](https://travis-ci.org/goby-lang/goby)
+[![Build Status](https://travis-ci.com/goby-lang/goby.svg?branch=master)](https://travis-ci.com/goby-lang/goby)
 [![GoDoc](https://godoc.org/github.com/goby-lang/goby?status.svg)](https://godoc.org/github.com/goby-lang/goby)
 [![Go Report Card](https://goreportcard.com/badge/github.com/goby-lang/goby)](https://goreportcard.com/report/github.com/goby-lang/goby)
 [![codecov](https://codecov.io/gh/goby-lang/goby/branch/master/graph/badge.svg)](https://codecov.io/gh/goby-lang/goby)

vm/http.go

@@ -28,6 +28,10 @@ var builtinHTTPClassMethods = []*BuiltinMethodObject{
 		// Sends a GET request to the target and returns the HTTP response as a string. Will error on non-200 responses, for more control over http requests look at the `start` method.
 		Name: "get",
 		Fn: func(receiver Object, sourceLine int, t *Thread, args []Object, blockFrame *normalCallFrame) Object {
+			if len(args) < 1 {
+				return t.vm.InitErrorObject(errors.ArgumentError, sourceLine, errors.WrongNumberOfArgumentMore, 1, len(args))
+			}
+
 			arg0, ok := args[0].(*StringObject)
 			if !ok {
 				return t.vm.InitErrorObject(errors.ArgumentError, sourceLine, errors.WrongArgumentTypeFormatNum, 0, "String", args[0].Class().Name)
@@ -115,6 +119,10 @@ var builtinHTTPClassMethods = []*BuiltinMethodObject{
 		// Sends a HEAD request to the target with type header and body. Returns the HTTP headers as a map[string]string. Will error on non-200 responses, for more control over http requests look at the `start` method.
 		Name: "head",
 		Fn: func(receiver Object, sourceLine int, t *Thread, args []Object, blockFrame *normalCallFrame) Object {
+			if len(args) < 1 {
+				return t.vm.InitErrorObject(errors.ArgumentError, sourceLine, errors.WrongNumberOfArgumentMore, 1, len(args))
+			}
+
 			arg0, ok := args[0].(*StringObject)
 			if !ok {
 				return t.vm.InitErrorObject(errors.ArgumentError, sourceLine, errors.WrongArgumentTypeFormatNum, 0, "String", args[0].Class().Name)

vm/http_test.go

@@ -77,6 +77,18 @@ func TestHTTPRequestFail(t *testing.T) {
 
 		Net::HTTP.get(42)
 		`, "ArgumentError: Expect argument #0 to be String. got: Integer", 1},
+		//Zero argument for get()
+		{`
+		require "net/http"
+
+		Net::HTTP.get
+		`, "ArgumentError: Expect 1 or more argument(s). got: 0", 1},
+		//Zero argument for head()
+		{`
+		require "net/http"
+
+		Net::HTTP.head
+		`, "ArgumentError: Expect 1 or more argument(s). got: 0", 1},
 		{`
 		require "net/http"
 

vm/uri.go

@@ -45,6 +45,7 @@ var builtinURIClassMethods = []*BuiltinMethodObject{
 				"@password": NULL,
 				"@query":    NULL,
 				"@path":     t.vm.InitStringObject("/"),
+				"@fragment": NULL,
 			}
 
 			// Scheme
@@ -92,6 +93,11 @@ var builtinURIClassMethods = []*BuiltinMethodObject{
 				}
 			}
 
+			// Fragment
+			if u.Fragment != "" {
+				uriAttrs["@fragment"] = t.vm.InitStringObject(u.Fragment)
+			}
+
 			var c *RClass
 
 			if u.Scheme == "https" {
@@ -134,6 +140,7 @@ func initURIClass(vm *VM) {
 		vm.InitStringObject("scheme"),
 		vm.InitStringObject("user"),
 		vm.InitStringObject("password"),
+		vm.InitStringObject("fragment"),
 	}
 
 	http.setAttrReader(attrs)

vm/uri_test.go

@@ -80,6 +80,12 @@ func TestURIParsing(t *testing.T) {
 		u = URI.parse("https://example.com")
 		u.password
 		`, nil},
+		// Fragment
+		{`
+		require "uri"
+		u = URI.parse("https://example.com#id1")
+		u.fragment
+		`, "id1"},
 	}
 
 	for i, tt := range tests {