Add specs for rb_time_timespec_new

Author: ahmadsherif

optional/capi/ext/rubyspec.h

@@ -596,6 +596,7 @@
 #define HAVE_RB_TIME_INTERVAL              1
 #define HAVE_RB_TIME_TIMEVAL               1
 #define HAVE_RB_TIME_TIMESPEC              1
+#define HAVE_RB_TIME_TIMESPEC_NEW          1
 
 /* Util */
 #define HAVE_RB_SCAN_ARGS                  1

optional/capi/ext/time_spec.c

@@ -55,6 +55,16 @@ static VALUE time_spec_rb_time_timespec(VALUE self, VALUE time) {
 }
 #endif
 
+#ifdef HAVE_RB_TIME_TIMESPEC_NEW
+static VALUE time_spec_rb_time_timespec_new(VALUE self, VALUE sec, VALUE nsec, VALUE offset) {
+  struct timespec ts;
+  ts.tv_sec = NUM2TIMET(sec);
+  ts.tv_nsec = NUM2LONG(nsec);
+
+  return rb_time_timespec_new(&ts, NUM2INT(offset));
+}
+#endif
+
 #ifdef HAVE_TIMET2NUM
 static VALUE time_spec_TIMET2NUM(VALUE self) {
   time_t t = 10;
@@ -93,6 +103,10 @@ void Init_time_spec() {
 #ifdef HAVE_RB_TIME_TIMESPEC
   rb_define_method(cls, "rb_time_timespec", time_spec_rb_time_timespec, 1);
 #endif
+
+#ifdef HAVE_RB_TIME_TIMESPEC_NEW
+  rb_define_method(cls, "rb_time_timespec_new", time_spec_rb_time_timespec_new, 3);
+#endif
 }
 
 #ifdef __cplusplus

optional/capi/time_spec.rb

@@ -251,4 +251,34 @@
       nsec.should == t.nsec
     end
   end
+
+  describe "rb_time_timespec_new" do
+    it "returns a time object with the given timespec and UTC offset" do
+      @s.rb_time_timespec_new(1447087832, 476451125, 32400).should == Time.at(1447087832, 476451.125).localtime(32400)
+    end
+
+    describe "when offset given is within range of -86400 and 86400 (exclusive)" do
+      it "sets time's is_gmt to false" do
+        @s.rb_time_timespec_new(1447087832, 476451125, 0).gmt?.should be_false
+      end
+
+      it "sets time's offset to the offset given" do
+        @s.rb_time_timespec_new(1447087832, 476451125, 86399).gmtoff.should == 86399
+      end
+    end
+
+    it "returns time object in UTC if offset given equals INT_MAX - 1" do
+      @s.rb_time_timespec_new(1447087832, 476451125, 0x7ffffffe).utc?.should be_true
+    end
+
+    it "returns time object in localtime if offset given equals INT_MAX" do
+      @s.rb_time_timespec_new(1447087832, 476451125, 0x7fffffff).should == Time.at(1447087832, 476451.125).localtime
+      @s.rb_time_timespec_new(1447087832, 476451125, 0x7fffffff).gmtoff.should == Time.now.gmtoff
+    end
+
+    it "raises an ArgumentError if offset passed is not within range of -86400 and 86400 (exclusive)" do
+      lambda { @s.rb_time_timespec_new(1447087832, 476451125, 86400) }.should raise_error(ArgumentError)
+      lambda { @s.rb_time_timespec_new(1447087832, 476451125, -86400) }.should raise_error(ArgumentError)
+    end
+  end
 end