Change how callers construct Clients to make testing easier. Then add tests for:

• [x] Posting
• [x] Fetching room history
• [x] Listing rooms

This addresses part of (1) in andybons/hipchat#1. Since we're adding tests anyways, I also added a Travis config file and badge.

This is the re-worked, leaner, meaner version of #2. It makes a few breaking changes, so it's tucked away in /v2/. Changes:

1. Client constructor takes a Config struct. This allows callers to specify just an auth token, but lets us alter the base URL in the test code.
2. Client constructor returns an interface, not a concrete type. This enables the next item.
3. Add a Mock that satisfies the Client interface. Callers can use the mock in their own test code.
4. Add tests for all methods on the real client and the mock. Per the built-in coverage tools, test coverage is a little over 93%.
5. Move the example into the test code, so it's compiler-verified and integrated into the docs.

• Return optionally structured errors with more information.

if hcErr, ok := err.(hipchat.Error); ok {
    log.Printf("[%d] %q", hcErr.Code, hcErr.Message)
}

• Actually return the error if a message wasn't posted.

This removes the human friendly error message being reported by https://github.com/andybons/hipchat/issues/8 and returns a standard Hipchat.Error instead.

I got this error when running your script from Debian 7 :

go run hipchat.go 
2015/10/22 09:22:41 Expected no error, but got "x509: certificate signed by unknown authority"

Script :

package main

import (
    "github.com/andybons/hipchat"
    "log"
)

func main() {
    c := hipchat.NewClient("xxxx")
    req := hipchat.MessageRequest{
        RoomId:        "xxxx",
        From:          "GoLang",
        Message:       "Bad news: Combustible lemons failed.",
        Color:         hipchat.ColorPurple,
        MessageFormat: hipchat.FormatText,
        Notify:        true,
    }

    if err := c.PostMessage(req); err != nil {
        log.Printf("Expected no error, but got %q", err)
    }
}

I keep getting the following when sending a message using the example:

2014/11/17 10:47:38 Expected no error, but got "PostMessage: response 'status' field was not 'sent'."

My example is as follows

package main

import (
    "github.com/andybons/hipchat"
    "log"
)

func main() {
    c := hipchat.Client{AuthToken: "xxxxxx"}
    req := hipchat.MessageRequest{
        RoomId:        "VCS Source",
        From:          "Aspect",
        Message:       "Bad news: Combustible lemons failed.",
        Color:         hipchat.ColorPurple,
        MessageFormat: hipchat.FormatText,
        Notify:        true,
    }

    if err := c.PostMessage(req); err != nil {
        log.Printf("Expected no error, but got %q", err)
    }
}

I have also tried an admin token, a room token, and a neither work

Hi Andy, let me know if you're interested in these changes. Keep in mind that this breaks the the PostMessage function. func (c *Client) PostMessage(m MessageRequest, h http.Client) error

I'm flexible on a lot of this.

Hey @andybons,

Thanks for this library! I'm using it for some internal tools, and it's been great. Two questions:

1. Are you open to a refactoring to make testing easier? I'd like to add both tests for this code and a mock Client implementation that callers can use in their tests.
2. I've written a polling client (runs in its own goroutine, polls for messages and pushes them onto a channel). I find it convenient - are you open to integrating it into this library as a second client type?

When hipchat-go client runs inside Private DC and need proxy settings to connect internet or particular development environment this settings are necessary

This will allow rooms/message API calls to use the 'auth_test' flag described here: https://www.hipchat.com/docs/api/auth

The response type is different than a normal request. Catch the 'success' vs 'error' response and return the error if present (nil if successfully authenticated).

Added a feature that allows you to specify a different endpoint Url when creating a Client struct. The code will use it if it is not blank. If it is the code defaults to the baseUrl.

In the middle of working on the v2 code we discussed, I realized that this project isn't open-source. Mind adding a license?

This PR adds the MIT license, but feel free to add whichever you'd like (or keep the code closed).